You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by ra...@apache.org on 2007/07/20 19:03:40 UTC
svn commit: r558052 -
/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQConnection.java
Author: rajdavies
Date: Fri Jul 20 10:03:39 2007
New Revision: 558052
URL: http://svn.apache.org/viewvc?view=rev&rev=558052
Log:
fix null pointer in test case
Modified:
activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQConnection.java
Modified: activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQConnection.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQConnection.java?view=diff&rev=558052&r1=558051&r2=558052
==============================================================================
--- activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQConnection.java (original)
+++ activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ActiveMQConnection.java Fri Jul 20 10:03:39 2007
@@ -1603,13 +1603,15 @@
}
@Override
- public Response processProducerAck(ProducerAck pa) throws Exception {
- ActiveMQMessageProducer producer = producers.get(pa.getProducerId());
- if( producer!=null ) {
- producer.onProducerAck(pa);
- }
- return null;
- }
+ public Response processProducerAck(ProducerAck pa) throws Exception{
+ if(pa!=null&&pa.getProducerId()!=null){
+ ActiveMQMessageProducer producer=producers.get(pa.getProducerId());
+ if(producer!=null){
+ producer.onProducerAck(pa);
+ }
+ }
+ return null;
+ }
@Override
public Response processBrokerInfo(BrokerInfo info) throws Exception {