Github user jdanekrh commented on the pull request:

    
https://github.com/apache/activemq-artemis/commit/576cf7e73ddabdcbd67ea24a9111889d6c4c7e48#commitcomment-29469395
  
    In 
artemis-server/src/main/java/org/apache/activemq/artemis/core/postoffice/impl/PostOfficeImpl.java:
    In 
artemis-server/src/main/java/org/apache/activemq/artemis/core/postoffice/impl/PostOfficeImpl.java
 on line 515:
    `user.equals(queue.getUser())` won't work. Variable `user` is a `String` 
and `queue.getUser()` gives `SimpleString`, so these will be never equal 
according to String#equals.
    
    and there is extra pair of parenthesis around the whole expression


---

Reply via email to