Repository: qpid-jms
Updated Branches:
  refs/heads/master b6d445f96 -> 9f0b49efc


NO-JIRA  Remove a couple old TODOs

Project: http://git-wip-us.apache.org/repos/asf/qpid-jms/repo
Commit: http://git-wip-us.apache.org/repos/asf/qpid-jms/commit/9f0b49ef
Tree: http://git-wip-us.apache.org/repos/asf/qpid-jms/tree/9f0b49ef
Diff: http://git-wip-us.apache.org/repos/asf/qpid-jms/diff/9f0b49ef

Branch: refs/heads/master
Commit: 9f0b49efcea805aa642b8f7d9601a22c3e7246e7
Parents: b6d445f
Author: Timothy Bish <[email protected]>
Authored: Mon Sep 28 18:08:24 2015 -0400
Committer: Timothy Bish <[email protected]>
Committed: Mon Sep 28 18:08:24 2015 -0400

----------------------------------------------------------------------
 .../qpid/jms/message/JmsMessagePropertyIntercepter.java      | 2 +-
 .../qpid/jms/provider/amqp/message/AmqpJmsMessageFacade.java | 8 --------
 2 files changed, 1 insertion(+), 9 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/9f0b49ef/qpid-jms-client/src/main/java/org/apache/qpid/jms/message/JmsMessagePropertyIntercepter.java
----------------------------------------------------------------------
diff --git 
a/qpid-jms-client/src/main/java/org/apache/qpid/jms/message/JmsMessagePropertyIntercepter.java
 
b/qpid-jms-client/src/main/java/org/apache/qpid/jms/message/JmsMessagePropertyIntercepter.java
index 4b7da1d..4dd3c13 100644
--- 
a/qpid-jms-client/src/main/java/org/apache/qpid/jms/message/JmsMessagePropertyIntercepter.java
+++ 
b/qpid-jms-client/src/main/java/org/apache/qpid/jms/message/JmsMessagePropertyIntercepter.java
@@ -419,7 +419,7 @@ public class JmsMessagePropertyIntercepter {
 
             @Override
             public void clearProperty(JmsMessageFacade message) {
-                message.setDeliveryCount(1);  // TODO - Delivery Count cleanup
+                message.setDeliveryCount(1);
             }
         });
         PROPERTY_INTERCEPTERS.put(JMSX_GROUPID, new PropertyIntercepter() {

http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/9f0b49ef/qpid-jms-client/src/main/java/org/apache/qpid/jms/provider/amqp/message/AmqpJmsMessageFacade.java
----------------------------------------------------------------------
diff --git 
a/qpid-jms-client/src/main/java/org/apache/qpid/jms/provider/amqp/message/AmqpJmsMessageFacade.java
 
b/qpid-jms-client/src/main/java/org/apache/qpid/jms/provider/amqp/message/AmqpJmsMessageFacade.java
index 090e775..893cb19 100644
--- 
a/qpid-jms-client/src/main/java/org/apache/qpid/jms/provider/amqp/message/AmqpJmsMessageFacade.java
+++ 
b/qpid-jms-client/src/main/java/org/apache/qpid/jms/provider/amqp/message/AmqpJmsMessageFacade.java
@@ -481,14 +481,6 @@ public class AmqpJmsMessageFacade implements 
JmsMessageFacade {
         setRedeliveryCount(deliveryCount - 1);
     }
 
-    // TODO - We can probably remove these set / get redelivery count and just 
use
-    //        the delivery count and is / set redelivered bits for the JMS 
mapping.
-    //
-    // possibly add an increment to make the consumer code more readable when 
doing
-    // a recover or rollback if we do local redeliveries.
-    //
-    //  public void incrementRedeliveryCount()
-
     @Override
     public int getRedeliveryCount() {
         if (message.getHeader() != null) {


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to