* Remove what should be unnecessary locking of GetScenePresences()

* May help with mantis 1434 though I doubt it

1 files changed, 3 insertions, 9 deletions

diff --git a/OpenSim/Region/Environment/Scenes/InnerScene.cs b/OpenSim/Region/Environment/Scenes/InnerScene.cs
index 3e250da..b6ba706 100644
--- a/OpenSim/Region/Environment/Scenes/InnerScene.cs
+++ b/OpenSim/Region/Environment/Scenes/InnerScene.cs
@@ -627,19 +627,13 @@ namespace OpenSim.Region.Environment.Scenes
         #region Get Methods
 
         /// <summary>
-        /// Request a List of all m_scenePresences in this World
+        /// Request a List of all scene presences in this scene.  This returns a copy of the original list, so no 
+        /// locking is required to iterate over it.
         /// </summary>
         /// <returns></returns>
         protected internal List<ScenePresence> GetScenePresences()
         {
-            List<ScenePresence> result;
-
-            lock (ScenePresences)
-            {
-                result = new List<ScenePresence>(ScenePresences.Values);
-            }
-
-            return result;
+            return new List<ScenePresence>(ScenePresences.Values);
         }
 
         protected internal List<ScenePresence> GetAvatars()