[AMQNET-518] lock(this.mutex)

• Type: Wish
• Status: Closed
• Priority: Trivial
• Resolution: Not A Problem
• Component/s: ActiveMQ, NMS
• Assignee: Jim Gomes
• Reporter: Artem Elkin
• Created: 10/Jan/16 02:56
• Updated: 15/Mar/16 14:23
• Resolved: 15/Mar/16 14:23

Description

Found in
http://svn.apache.org/viewvc/activemq/activemq-dotnet/Apache.NMS.ActiveMQ/branches/1.7.x/src/main/csharp/Util/FifoMessageDispatchChannel.cs?view=markup
and in some other places:

private readonly Mutex mutex = new Mutex();
...
lock(this.mutex)
...
Monitor.PulseAll(this.mutex);
...
etc.

Guys, do you really know .Net? ;-D
Monitor and lock (which uses Monitor internally) work for any reference type instances. Please change the field type to object and change the name too. This is misleading now.