svn commit: r1080847 - /commons/proper/net/trunk/src/main/java/org/apache/commons/net/nntp/NNTPClient.java

[sebb]

Author: sebb
Date: Sat Mar 12 02:09:42 2011
New Revision: 1080847

URL: http://svn.apache.org/viewvc?rev=1080847&view=rev
Log:
Align with other iterate methods and throw IOException on failure

Modified:
    
commons/proper/net/trunk/src/main/java/org/apache/commons/net/nntp/NNTPClient.java

Modified: 
commons/proper/net/trunk/src/main/java/org/apache/commons/net/nntp/NNTPClient.java
URL: 
http://svn.apache.org/viewvc/commons/proper/net/trunk/src/main/java/org/apache/commons/net/nntp/NNTPClient.java?rev=1080847&r1=1080846&r2=1080847&view=diff
==============================================================================
--- 
commons/proper/net/trunk/src/main/java/org/apache/commons/net/nntp/NNTPClient.java
 (original)
+++ 
commons/proper/net/trunk/src/main/java/org/apache/commons/net/nntp/NNTPClient.java
 Sat Mar 12 02:09:42 2011
@@ -1419,8 +1419,8 @@ public class NNTPClient extends NNTP
      * <p>
      * @param lowArticleNumber
      * @param highArticleNumber
-     * @return an Iterable of Articles, or {@code null} if the command failed
-     * @throws IOException
+     * @return an Iterable of Articles
+     * @throws IOException if the command failed
      * @since 3.0
      */
     public Iterable<Article> iterateArticleInfo(long lowArticleNumber, long 
highArticleNumber) 
@@ -1428,7 +1428,7 @@ public class NNTPClient extends NNTP
     {
         Reader info = retrieveArticleInfo(lowArticleNumber,highArticleNumber);
         if (info == null) {
-            return null;
+            throw new IOException(getReplyString());
         }
         // N.B. info is already DotTerminated, so don't rewrap
         return new ArticleIterator(new ReplyIterator(info, false));