You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by pm...@apache.org on 2016/12/28 21:33:45 UTC
svn commit: r1776331 -
/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/client/ReceiveSubscriber.java
Author: pmouawad
Date: Wed Dec 28 21:33:45 2016
New Revision: 1776331
URL: http://svn.apache.org/viewvc?rev=1776331&view=rev
Log:
sonar: fix errors
Modified:
jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/client/ReceiveSubscriber.java
Modified: jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/client/ReceiveSubscriber.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/client/ReceiveSubscriber.java?rev=1776331&r1=1776330&r2=1776331&view=diff
==============================================================================
--- jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/client/ReceiveSubscriber.java (original)
+++ jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/client/ReceiveSubscriber.java Wed Dec 28 21:33:45 2016
@@ -329,6 +329,7 @@ public class ReceiveSubscriber implement
}
} catch (InterruptedException e) {
// Ignored
+ Thread.currentThread().interrupt();
}
return message;
}
@@ -352,7 +353,7 @@ public class ReceiveSubscriber implement
connectionStarted = false;
}
} catch (JMSException e) {
- log.warn("Stopping connection throws exception, message:"+e.getMessage());
+ log.warn("Stopping connection throws exception, message:"+e.getMessage(), e);
}
Utils.close(subscriber, log);
Utils.close(session, log);
@@ -378,6 +379,6 @@ public class ReceiveSubscriber implement
* @return True if input is null, an empty string, or a white space-only string
*/
private boolean isEmpty(String s1) {
- return (s1 == null || s1.trim().isEmpty());
+ return s1 == null || s1.trim().isEmpty();
}
}