MSDN documentation is unclear about whether exiting a lock() block will trigger a Monitor.Wait() to exit, so avoid some locks that don't actually affect the state of the internal queues in the BlockingQueue class.

cpu-performance
dahlia 2013-07-16 01:12:56 -07:00
parent e8e073aa97
commit 42e2a0d66e
1 changed files with 8 additions and 5 deletions

View File

@ -58,7 +58,7 @@ namespace OpenSim.Framework
{ {
lock (m_queueSync) lock (m_queueSync)
{ {
if (m_queue.Count < 1 && m_pqueue.Count < 1) while (m_queue.Count < 1 && m_pqueue.Count < 1)
{ {
Monitor.Wait(m_queueSync); Monitor.Wait(m_queueSync);
} }
@ -91,6 +91,9 @@ namespace OpenSim.Framework
public bool Contains(T item) public bool Contains(T item)
{ {
if (m_queue.Count < 1 && m_pqueue.Count < 1)
return false;
lock (m_queueSync) lock (m_queueSync)
{ {
if (m_pqueue.Contains(item)) if (m_pqueue.Contains(item))
@ -100,15 +103,15 @@ namespace OpenSim.Framework
} }
public int Count() public int Count()
{
lock (m_queueSync)
{ {
return m_queue.Count+m_pqueue.Count; return m_queue.Count+m_pqueue.Count;
} }
}
public T[] GetQueueArray() public T[] GetQueueArray()
{ {
if (m_queue.Count < 1 && m_pqueue.Count < 1)
return new T[0];
lock (m_queueSync) lock (m_queueSync)
{ {
return m_queue.ToArray(); return m_queue.ToArray();