You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by no...@apache.org on 2011/01/09 17:55:53 UTC

svn commit: r1056976 - /james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/jms/JMSMailQueue.java

Author: norman
Date: Sun Jan  9 16:55:52 2011
New Revision: 1056976

URL: http://svn.apache.org/viewvc?rev=1056976&view=rev
Log:
Only some cleanup

Modified:
    james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/jms/JMSMailQueue.java

Modified: james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/jms/JMSMailQueue.java
URL: http://svn.apache.org/viewvc/james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/jms/JMSMailQueue.java?rev=1056976&r1=1056975&r2=1056976&view=diff
==============================================================================
--- james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/jms/JMSMailQueue.java (original)
+++ james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/jms/JMSMailQueue.java Sun Jan  9 16:55:52 2011
@@ -760,10 +760,17 @@ public class JMSMailQueue implements Man
             
             return new MailQueueIterator() {
                 
+                /**
+                 * Not supported, read-only
+                 */
                 public void remove() {
                     throw new UnsupportedOperationException("Read-only");
                 }
                 
+                /*
+                 * (non-Javadoc)
+                 * @see java.util.Iterator#next()
+                 */
                 public Mail next() {
                     while (hasNext()) {
                         try {
@@ -779,6 +786,10 @@ public class JMSMailQueue implements Man
                     
                 }
                 
+                /*
+                 * (non-Javadoc)
+                 * @see java.util.Iterator#hasNext()
+                 */
                 public boolean hasNext() {
                     return messages.hasMoreElements();
                 }
@@ -830,8 +841,8 @@ public class JMSMailQueue implements Man
             } catch (JMSException e1) {
                 // ignore here
             }
-            logger.error("Unable to get size of queue " + queuename, e);
-            throw new MailQueueException("Unable to get size of queue " + queuename, e);
+            logger.error("Unable to browse queue " + queuename, e);
+            throw new MailQueueException("Unable to browse queue " + queuename, e);
         }
     }
 



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