Slight variation on the locking scheme: now locking always, except the ForEach, which gets a copy of the Array. I think the ForEach was the big gorilla.

1 files changed, 9 insertions, 9 deletions

diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLUDPClientCollection.cs b/OpenSim/Region/ClientStack/LindenUDP/LLUDPClientCollection.cs
index 784426f..dbb9861 100644
--- a/OpenSim/Region/ClientStack/LindenUDP/LLUDPClientCollection.cs
+++ b/OpenSim/Region/ClientStack/LindenUDP/LLUDPClientCollection.cs
@@ -247,16 +247,16 @@ namespace OpenSim.Region.ClientStack.LindenUDP
 
             ////return success;
 
-            //lock (m_sync)
-            //    return Dictionary2.TryGetValue(key, out value);
-
-            try
-            {
+            lock (m_sync)
                 return Dictionary2.TryGetValue(key, out value);
-            }
-            catch { }
-            value = null;
-            return false;
+
+            //try
+            //{
+            //    return Dictionary2.TryGetValue(key, out value);
+            //}
+            //catch { }
+            //value = null;
+            //return false;
 
         }