You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@activemq.apache.org by da...@apache.org on 2012/09/08 12:27:19 UTC

svn commit: r1382275 - /activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQMessageProducer.java

Author: davsclaus
Date: Sat Sep  8 10:27:19 2012
New Revision: 1382275

URL: http://svn.apache.org/viewvc?rev=1382275&view=rev
Log:
AMQ-3976: Use equals instead of == in checking destination names. Thanks to Lior for the patch.

Modified:
    activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQMessageProducer.java

Modified: activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQMessageProducer.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQMessageProducer.java?rev=1382275&r1=1382274&r2=1382275&view=diff
==============================================================================
--- activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQMessageProducer.java (original)
+++ activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQMessageProducer.java Sat Sep  8 10:27:19 2012
@@ -248,7 +248,7 @@ public class ActiveMQMessageProducer ext
         }
 
         ActiveMQDestination dest;
-        if (destination == info.getDestination()) {
+        if (destination.equals(info.getDestination())) {
             dest = (ActiveMQDestination)destination;
         } else if (info.getDestination() == null) {
             dest = ActiveMQDestination.transform(destination);