Uploaded image for project: 'ActiveMQ .Net'
  1. ActiveMQ .Net
  2. AMQNET-518

lock(this.mutex)

    XMLWordPrintableJSON

    Details

    • Type: Wish
    • Status: Closed
    • Priority: Trivial
    • Resolution: Not A Problem
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: ActiveMQ, NMS
    • Labels:
      None

      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.

        Attachments

          Activity

            People

            • Assignee:
              jgomes Jim Gomes
              Reporter:
              artelk Artem Elkin
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: