You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ra...@apache.org on 2013/07/23 16:24:26 UTC

svn commit: r1506055 - in /qpid/trunk/qpid/java/client/src: main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java test/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10Test.java

Author: rajith
Date: Tue Jul 23 14:24:25 2013
New Revision: 1506055

URL: http://svn.apache.org/r1506055
Log:
NO_JIRA Removed two System.out's from the code.

Modified:
    qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java
    qpid/trunk/qpid/java/client/src/test/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10Test.java

Modified: qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java?rev=1506055&r1=1506054&r2=1506055&view=diff
==============================================================================
--- qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java (original)
+++ qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java Tue Jul 23 14:24:25 2013
@@ -125,7 +125,6 @@ public class AMQMessageDelegate_0_10 ext
             String subject = null;
             if (isStrictJMS && messageProps != null && messageProps.getApplicationHeaders() != null)
             {
-                System.out.println("%%%% Going to set subject");
                 subject = (String)messageProps.getApplicationHeaders().get(QpidMessageProperties.QPID_SUBJECT);
                 if (subject != null)
                 {

Modified: qpid/trunk/qpid/java/client/src/test/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10Test.java
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/java/client/src/test/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10Test.java?rev=1506055&r1=1506054&r2=1506055&view=diff
==============================================================================
--- qpid/trunk/qpid/java/client/src/test/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10Test.java (original)
+++ qpid/trunk/qpid/java/client/src/test/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10Test.java Tue Jul 23 14:24:25 2013
@@ -124,7 +124,6 @@ public class AMQMessageDelegate_0_10Test
             for (Enumeration props = delegate.getPropertyNames(); props.hasMoreElements();)
             {
                 String key = (String)props.nextElement();
-                System.out.println("PropName : " + key);
                 if (key.equals("JMS_" + QpidMessageProperties.QPID_SUBJECT))
                 {
                     propFound = true;



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org


Re: svn commit: r1506055 - in /qpid/trunk/qpid/java/client/src: main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java test/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10Test.java

Posted by Robbie Gemmell <ro...@gmail.com>.
Since I just picked on Ted, it seems only fair to share the love... :)

This shouldn't really have been a NO-JIRA either (or rather, a NO_JIRA in
this case). When I pointed this out to you I was replying to the commit for
QPID-3838 that added them, so it seems reasonable that this commit and the
merge to 0.24 were actually also for QPID-3838.

Robbie

On 23 July 2013 15:24, <ra...@apache.org> wrote:

> Author: rajith
> Date: Tue Jul 23 14:24:25 2013
> New Revision: 1506055
>
> URL: http://svn.apache.org/r1506055
> Log:
> NO_JIRA Removed two System.out's from the code.
>
> Modified:
>
> qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java
>
> qpid/trunk/qpid/java/client/src/test/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10Test.java
>
> Modified:
> qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java
> URL:
> http://svn.apache.org/viewvc/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java?rev=1506055&r1=1506054&r2=1506055&view=diff
>
> ==============================================================================
> ---
> qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java
> (original)
> +++
> qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java
> Tue Jul 23 14:24:25 2013
> @@ -125,7 +125,6 @@ public class AMQMessageDelegate_0_10 ext
>              String subject = null;
>              if (isStrictJMS && messageProps != null &&
> messageProps.getApplicationHeaders() != null)
>              {
> -                System.out.println("%%%% Going to set subject");
>                  subject =
> (String)messageProps.getApplicationHeaders().get(QpidMessageProperties.QPID_SUBJECT);
>                  if (subject != null)
>                  {
>
> Modified:
> qpid/trunk/qpid/java/client/src/test/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10Test.java
> URL:
> http://svn.apache.org/viewvc/qpid/trunk/qpid/java/client/src/test/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10Test.java?rev=1506055&r1=1506054&r2=1506055&view=diff
>
> ==============================================================================
> ---
> qpid/trunk/qpid/java/client/src/test/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10Test.java
> (original)
> +++
> qpid/trunk/qpid/java/client/src/test/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10Test.java
> Tue Jul 23 14:24:25 2013
> @@ -124,7 +124,6 @@ public class AMQMessageDelegate_0_10Test
>              for (Enumeration props = delegate.getPropertyNames();
> props.hasMoreElements();)
>              {
>                  String key = (String)props.nextElement();
> -                System.out.println("PropName : " + key);
>                  if (key.equals("JMS_" +
> QpidMessageProperties.QPID_SUBJECT))
>                  {
>                      propFound = true;
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
> For additional commands, e-mail: commits-help@qpid.apache.org
>
>