No need to synchronise MoveNext because it only delegates to Next

michaelklishin · michaelklishin · commit 1ec2743cf6b5 · 2014-04-16T13:08:05.000+04:00
diff --git a/projects/client/RabbitMQ.Client/src/client/messagepatterns/Subscription.cs b/projects/client/RabbitMQ.Client/src/client/messagepatterns/Subscription.cs
@@ -420,10 +420,7 @@ object IEnumerator.Current {
         ///</remarks>
         bool IEnumerator.MoveNext()
         {
-            lock(m_eventLock)
-            {
-                return Next() != null;
-            }
+            return Next() != null;
         }
 
         ///<summary>Dummy implementation of the IEnumerator interface,

Original file line number	Diff line number	Diff line change
`@@ -420,10 +420,7 @@ object IEnumerator.Current {`
`420`	`420`	`///</remarks>`
`421`	`421`	`bool IEnumerator.MoveNext()`
`422`	`422`	`{`
`423`		`- lock(m_eventLock)`
`424`		`- {`
`425`		`- return Next() != null;`
`426`		`- }`
	`423`	`+ return Next() != null;`
`427`	`424`	`}`
`428`	`425`
`429`	`426`	`///<summary>Dummy implementation of the IEnumerator interface,`