Move SenseRepeaters.Count check inside the SenseRepeatListLock.

No methods in the List class are thread safe in the MS specification/documentation
2012-03-02 00:28:37 +00:00 · 2012-03-02 00:28:37 +00:00 · a315aee803
parent 0c931e3657
commit a315aee803
1 changed files with 4 additions and 4 deletions
--- a/OpenSim/Region/ScriptEngine/Shared/Api/Implementation/Plugins/SensorRepeat.cs
+++ b/OpenSim/Region/ScriptEngine/Shared/Api/Implementation/Plugins/SensorRepeat.cs
@ -151,13 +151,13 @@ namespace OpenSim.Region.ScriptEngine.Shared.Api.Plugins
        }

        public void CheckSenseRepeaterEvents()
+        {
+            lock (SenseRepeatListLock)
            {
                // Nothing to do here?
                if (SenseRepeaters.Count == 0)
                    return;

-            lock (SenseRepeatListLock)
-            {
                // Go through all timers
                foreach (SenseRepeatClass ts in SenseRepeaters)
                {