1 Reply Latest reply on Aug 11, 2008 5:47 AM by Jeff Mesnil

    Some duplicated code in MessageReferenceImpl

    Tim Fox Master

      I notice that a move() method was added, but the methods cancel and sendToDLQ seem to be doing the same thing. Shouldn't they call move?

      Also

      Please avoid

      if (blah)
      {
       ...
      } else
      {
       ...
      }
      

      Instead use:

      if (blah)
      {
       ...
      }
      else
      {
       ...
      }