[jboss-dev-forums] [Design of Messaging on JBoss (Messaging/JBoss)] - Some duplicated code in MessageReferenceImpl

timfox do-not-reply at jboss.com
Sat Aug 9 08:00:15 EDT 2008


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
  | {
  |    ...
  | }
  | 

View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4169724#4169724

Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4169724



More information about the jboss-dev-forums mailing list