[jboss-cvs] JBoss Messaging SVN: r2505 - in trunk: tests/src/org/jboss/test/messaging/jms and 1 other directory.

[jboss-cvs-commits at lists.jboss.org]

Author: ovidiu.feodorov at jboss.com
Date: 2007-02-28 18:39:41 -0500 (Wed, 28 Feb 2007)
New Revision: 2505

Modified:
   trunk/docs/examples/common/src/org/jboss/example/jms/common/ExampleSupport.java
   trunk/docs/examples/common/src/org/jboss/example/jms/common/Util.java
   trunk/tests/src/org/jboss/test/messaging/jms/QueueTest.java
Log:
minor

Modified: trunk/docs/examples/common/src/org/jboss/example/jms/common/ExampleSupport.java
===================================================================
--- trunk/docs/examples/common/src/org/jboss/example/jms/common/ExampleSupport.java	2007-02-28 23:09:29 UTC (rev 2504)
+++ trunk/docs/examples/common/src/org/jboss/example/jms/common/ExampleSupport.java	2007-02-28 23:39:41 UTC (rev 2505)
@@ -184,8 +184,6 @@
       setup(null);
    }
    
-   
-   
    protected void setup(InitialContext ic) throws Exception
    {
       String destinationName;

Modified: trunk/docs/examples/common/src/org/jboss/example/jms/common/Util.java
===================================================================
--- trunk/docs/examples/common/src/org/jboss/example/jms/common/Util.java	2007-02-28 23:09:29 UTC (rev 2504)
+++ trunk/docs/examples/common/src/org/jboss/example/jms/common/Util.java	2007-02-28 23:39:41 UTC (rev 2505)
@@ -66,16 +66,17 @@
    {
        deployQueue(jndiName,null);
    }
-   
+
+   // TODO: NOT TESTED!
    public static void activateMessagePullPolicy(InitialContext ic) throws Exception
    {
-      //Need to promgrammatically activate the default message pull policy.
+      // Need to promgrammatically activate the default message pull policy.
       
-      //We need to do this here since the default config ships with the NullMessagePullPolicy which
-      //doesn't do message redistribution
+      // We need to do this here since the default config ships with the NullMessagePullPolicy which
+      // doesn't do message redistribution.
       
-      //You won't have to do this in your own programs - you just need to make sure
-      //your postoffice MBean config specifies the DefaultMessagePullPolicy
+      // You won't have to do this in your own programs - you just need to make sure your postoffice
+      // MBean config specifies the DefaultMessagePullPolicy
       
       MBeanServerConnection mBeanServer = lookupMBeanServerProxy(ic);
       
@@ -83,15 +84,15 @@
 
       mBeanServer.invoke(postOfficeObjectName, "stop", null, null);
       
-      Attribute att = new Attribute("MessagePullPolicy", "org.jboss.messaging.core.plugin.postoffice.cluster.NullMessagePullPolicy");
+      Attribute att =
+         new Attribute("MessagePullPolicy",
+                       "org.jboss.messaging.core.plugin.postoffice.cluster.NullMessagePullPolicy");
       
       mBeanServer.setAttribute(postOfficeObjectName, att);
 
-      //Restart the post office
+      // Restart the post office
       
       mBeanServer.invoke(postOfficeObjectName, "start", null, null);
-      
-      
    }
 
    public static void deployQueue(String jndiName, InitialContext ic) throws Exception

Modified: trunk/tests/src/org/jboss/test/messaging/jms/QueueTest.java
===================================================================
--- trunk/tests/src/org/jboss/test/messaging/jms/QueueTest.java	2007-02-28 23:09:29 UTC (rev 2504)
+++ trunk/tests/src/org/jboss/test/messaging/jms/QueueTest.java	2007-02-28 23:39:41 UTC (rev 2505)
@@ -31,7 +31,6 @@
 import javax.naming.InitialContext;
 
 import org.jboss.jms.client.JBossConnectionFactory;
-import org.jboss.jms.client.JBossConnection;
 import org.jboss.test.messaging.MessagingTestCase;
 import org.jboss.test.messaging.tools.ServerManagement;