You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jmeter-dev@jakarta.apache.org by se...@apache.org on 2005/11/13 23:26:57 UTC

svn commit: r343988 - /jakarta/jmeter/branches/rel-2-1/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/JMSSampler.java

Author: sebb
Date: Sun Nov 13 14:26:52 2005
New Revision: 343988

URL: http://svn.apache.org/viewcvs?rev=343988&view=rev
Log:
Use entrySet iterator instead of keySet + get

Modified:
    jakarta/jmeter/branches/rel-2-1/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/JMSSampler.java

Modified: jakarta/jmeter/branches/rel-2-1/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/JMSSampler.java
URL: http://svn.apache.org/viewcvs/jakarta/jmeter/branches/rel-2-1/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/JMSSampler.java?rev=343988&r1=343987&r2=343988&view=diff
==============================================================================
--- jakarta/jmeter/branches/rel-2-1/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/JMSSampler.java (original)
+++ jakarta/jmeter/branches/rel-2-1/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/JMSSampler.java Sun Nov 13 14:26:52 2005
@@ -55,27 +55,27 @@
  */
 public class JMSSampler extends AbstractSampler {
 
-	public static final String JNDI_INITIAL_CONTEXT_FACTORY = "JMSSampler.initialContextFactory";
+	public static final String JNDI_INITIAL_CONTEXT_FACTORY = "JMSSampler.initialContextFactory"; // $NON-NLS-1$
 
-	public static final String JNDI_CONTEXT_PROVIDER_URL = "JMSSampler.contextProviderUrl";
+	public static final String JNDI_CONTEXT_PROVIDER_URL = "JMSSampler.contextProviderUrl"; // $NON-NLS-1$
 
-	public static final String JNDI_PROPERTIES = "JMSSampler.jndiProperties";
+	public static final String JNDI_PROPERTIES = "JMSSampler.jndiProperties"; // $NON-NLS-1$
 
 	private static final int DEFAULT_TIMEOUT = 2000;
 
-	public final static String TIMEOUT = "JMSSampler.timeout";
+	public final static String TIMEOUT = "JMSSampler.timeout"; // $NON-NLS-1$
 
-	public static final String IS_ONE_WAY = "JMSSampler.isFireAndForget";
+	public static final String IS_ONE_WAY = "JMSSampler.isFireAndForget"; // $NON-NLS-1$
 
-	public static final String JMS_PROPERTIES = "arguments";
+	public static final String JMS_PROPERTIES = "arguments"; // $NON-NLS-1$
 
-	public static final String RECEIVE_QUEUE = "JMSSampler.ReceiveQueue";
+	public static final String RECEIVE_QUEUE = "JMSSampler.ReceiveQueue"; // $NON-NLS-1$
 
-	public static final String XML_DATA = "HTTPSamper.xml_data";
+	public static final String XML_DATA = "HTTPSamper.xml_data"; // $NON-NLS-1$
 
-	public final static String SEND_QUEUE = "JMSSampler.SendQueue";
+	public final static String SEND_QUEUE = "JMSSampler.SendQueue"; // $NON-NLS-1$
 
-	public final static String QUEUE_CONNECTION_FACTORY_JNDI = "JMSSampler.queueconnectionfactory";
+	public final static String QUEUE_CONNECTION_FACTORY_JNDI = "JMSSampler.queueconnectionfactory"; // $NON-NLS-1$
 
 	private static final Logger LOGGER = LoggingManager.getLoggerForClass();
 
@@ -166,10 +166,11 @@
 
 	private void addJMSProperties(TextMessage msg) throws JMSException {
 		Map map = getArguments(JMSSampler.JMS_PROPERTIES).getArgumentsAsMap();
-		Iterator argIt = map.keySet().iterator();
+		Iterator argIt = map.entrySet().iterator();
 		while (argIt.hasNext()) {
-			String name = (String) argIt.next();
-			String value = (String) map.get(name);
+            Map.Entry me = (Map.Entry) argIt.next();
+			String name = (String) me.getKey();
+			String value = (String) me.getValue();
 			if (LOGGER.isDebugEnabled()) {
 				LOGGER.debug("Adding property [" + name + "=" + value + "]");
 			}



---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org