[prev in list] [next in list] [prev in thread] [next in thread] 

List:       jakarta-commons-dev
Subject:    svn commit: r523750 - in
From:       rwinston () apache ! org
Date:       2007-03-29 15:37:01
Message-ID: 20070329153702.465CE1A9838 () eris ! apache ! org
[Download RAW message or body]

Author: rwinston
Date: Thu Mar 29 08:37:00 2007
New Revision: 523750

URL: http://svn.apache.org/viewvc?view=rev&rev=523750
Log:
Fix NET-155

Modified:
    jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/examples/nntp/ExtendedNNTPOps.java
  jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/examples/nntp/MessageThreading.java
  jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/examples/nntp/NNTPUtils.java
  jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/org/apache/commons/net/nntp/NNTPClient.java
  jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/org/apache/commons/net/nntp/NewsgroupInfo.java
  jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/site/xdoc/changes.xml

Modified: jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/examples/nntp/ExtendedNNTPOps.java
                
URL: http://svn.apache.org/viewvc/jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/s \
rc/main/java/examples/nntp/ExtendedNNTPOps.java?view=diff&rev=523750&r1=523749&r2=523750
 ==============================================================================
--- jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/examples/nntp/ExtendedNNTPOps.java \
                (original)
+++ jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/examples/nntp/ExtendedNNTPOps.java \
Thu Mar 29 08:37:00 2007 @@ -57,8 +57,8 @@
 			// XOVER
 			NewsgroupInfo testGroup = new NewsgroupInfo();
 			client.selectNewsgroup("alt.test", testGroup);
-			int lowArticleNumber = testGroup.getFirstArticle();
-			int highArticleNumber = lowArticleNumber + 100;
+			long lowArticleNumber = testGroup.getFirstArticle();
+			long  highArticleNumber = lowArticleNumber + 100;
 			Article[] articles = NNTPUtils.getArticleInfo(client, lowArticleNumber, \
highArticleNumber);  
 			for (int i = 0; i < articles.length; ++i) {

Modified: jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/examples/nntp/MessageThreading.java
                
URL: http://svn.apache.org/viewvc/jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/s \
rc/main/java/examples/nntp/MessageThreading.java?view=diff&rev=523750&r1=523749&r2=523750
 ==============================================================================
--- jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/examples/nntp/MessageThreading.java \
                (original)
+++ jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/examples/nntp/MessageThreading.java \
Thu Mar 29 08:37:00 2007 @@ -54,8 +54,8 @@
 		NewsgroupInfo group = new NewsgroupInfo();
 		client.selectNewsgroup("comp.lang.lisp", group);
 		
-		int lowArticleNumber = group.getFirstArticle();
-		int highArticleNumber = lowArticleNumber + 100;
+		long lowArticleNumber = group.getFirstArticle();
+		long highArticleNumber = lowArticleNumber + 100;
 		
 		System.out.println("Retrieving articles between [" + lowArticleNumber + "] and [" \
+ highArticleNumber + "]");  Article[] articles = NNTPUtils.getArticleInfo(client, \
lowArticleNumber, highArticleNumber);

Modified: jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/examples/nntp/NNTPUtils.java
                
URL: http://svn.apache.org/viewvc/jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/s \
rc/main/java/examples/nntp/NNTPUtils.java?view=diff&rev=523750&r1=523749&r2=523750 \
                ==============================================================================
                
--- jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/examples/nntp/NNTPUtils.java \
                (original)
+++ jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/examples/nntp/NNTPUtils.java \
Thu Mar 29 08:37:00 2007 @@ -42,7 +42,7 @@
 	 * @return Article[] An array of Article
 	 * @throws IOException
 	 */
-	public  static Article[] getArticleInfo(NNTPClient client, int lowArticleNumber, \
int highArticleNumber) +	public  static Article[] getArticleInfo(NNTPClient client, \
long lowArticleNumber, long highArticleNumber)  throws IOException {
 			Reader reader = null;
 			Article[] articles = null;

Modified: jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/org/apache/commons/net/nntp/NNTPClient.java
                
URL: http://svn.apache.org/viewvc/jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/s \
rc/main/java/org/apache/commons/net/nntp/NNTPClient.java?view=diff&rev=523750&r1=523749&r2=523750
 ==============================================================================
--- jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/org/apache/commons/net/nntp/NNTPClient.java \
                (original)
+++ jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/org/apache/commons/net/nntp/NNTPClient.java \
Thu Mar 29 08:37:00 2007 @@ -176,7 +176,7 @@
     {
         NewsgroupInfo result;
         StringTokenizer tokenizer;
-        int lastNum, firstNum;
+        long lastNum, firstNum;
         String last, first, permission;
 
         result = new NewsgroupInfo();
@@ -192,8 +192,8 @@
 
         try
         {
-            lastNum = Integer.parseInt(last);
-            firstNum = Integer.parseInt(first);
+            lastNum = Long.valueOf(last);
+            firstNum = Long.valueOf(first);
             result._setFirstArticle(firstNum);
             result._setLastArticle(lastNum);
 
@@ -1212,8 +1212,8 @@
      * @return a DotTerminatedReader if successful, null otherwise
      * @throws IOException
      */
-    public Reader retrieveArticleInfo(int lowArticleNumber,
-                                      int highArticleNumber)
+    public Reader retrieveArticleInfo(long lowArticleNumber,
+                                      long highArticleNumber)
         throws IOException
     {
         return

Modified: jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/org/apache/commons/net/nntp/NewsgroupInfo.java
                
URL: http://svn.apache.org/viewvc/jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/s \
rc/main/java/org/apache/commons/net/nntp/NewsgroupInfo.java?view=diff&rev=523750&r1=523749&r2=523750
 ==============================================================================
--- jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/org/apache/commons/net/nntp/NewsgroupInfo.java \
                (original)
+++ jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/main/java/org/apache/commons/net/nntp/NewsgroupInfo.java \
Thu Mar 29 08:37:00 2007 @@ -32,124 +32,118 @@
  * @see NNTPClient
  ***/
 
-public final class NewsgroupInfo
-{
-    /***
-     * A constant indicating that the posting permission of a newsgroup is
-     * unknown.  For example, the NNTP GROUP command does not return posting
-     * information, so NewsgroupInfo instances obtained from that command
-     * willhave an UNKNOWN_POSTING_PERMISSION.
-     ***/
-    public static final int UNKNOWN_POSTING_PERMISSION = 0;
-
-    /*** A constant indicating that a newsgroup is moderated. ***/
-    public static final int MODERATED_POSTING_PERMISSION = 1;
-
-    /*** A constant indicating that a newsgroup is public and unmoderated. ***/
-    public static final int PERMITTED_POSTING_PERMISSION = 2;
-
-    /***
-     * A constant indicating that a newsgroup is closed for general posting.
-     ***/
-    public static final int PROHIBITED_POSTING_PERMISSION = 3;
-
-    private String __newsgroup;
-    private int __estimatedArticleCount;
-    private int __firstArticle, __lastArticle;
-    private int __postingPermission;
-
-    void _setNewsgroup(String newsgroup)
-    {
-        __newsgroup = newsgroup;
-    }
-
-    void _setArticleCount(int count)
-    {
-        __estimatedArticleCount = count;
-    }
-
-    void _setFirstArticle(int first)
-    {
-        __firstArticle = first;
-    }
-
-    void _setLastArticle(int last)
-    {
-        __lastArticle = last;
-    }
-
-    void _setPostingPermission(int permission)
-    {
-        __postingPermission = permission;
-    }
-
-    /***
-     * Get the newsgroup name.
-     * <p>
-     * @return The name of the newsgroup.
-     ***/
-    public String getNewsgroup()
-    {
-        return __newsgroup;
-    }
-
-    /***
-     * Get the estimated number of articles in the newsgroup.  The
-     * accuracy of this value will depend on the server implementation.
-     * <p>
-     * @return The estimated number of articles in the newsgroup.
-     ***/
-    public int getArticleCount()
-    {
-        return __estimatedArticleCount;
-    }
-
-    /***
-     * Get the number of the first article in the newsgroup.
-     * <p>
-     * @return The number of the first article in the newsgroup.
-     ***/
-    public int getFirstArticle()
-    {
-        return __firstArticle;
-    }
-
-    /***
-     * Get the number of the last article in the newsgroup.
-     * <p>
-     * @return The number of the last article in the newsgroup.
-     ***/
-    public int getLastArticle()
-    {
-        return __lastArticle;
-    }
-
-    /***
-     * Get the posting permission of the newsgroup.  This will be one of
-     * the <code> POSTING_PERMISSION </code> constants.
-     * <p>
-     * @return The posting permission status of the newsgroup.
-     ***/
-    public int getPostingPermission()
-    {
-        return __postingPermission;
-    }
-
-    /*
-    public String toString() {
-      StringBuffer buffer = new StringBuffer();
-      buffer.append(__newsgroup);
-      buffer.append(' ');
-      buffer.append(__lastArticle);
-      buffer.append(' ');
-      buffer.append(__firstArticle);
-      buffer.append(' ');
-      switch(__postingPermission) {
-        case 1: buffer.append('m'); break;
-        case 2: buffer.append('y'); break;
-        case 3: buffer.append('n'); break;
-      }
-      return buffer.toString();
-}
-    */
+public final class NewsgroupInfo {
+	/***
+	 * A constant indicating that the posting permission of a newsgroup is
+	 * unknown.  For example, the NNTP GROUP command does not return posting
+	 * information, so NewsgroupInfo instances obtained from that command
+	 * willhave an UNKNOWN_POSTING_PERMISSION.
+	 ***/
+	public static final int UNKNOWN_POSTING_PERMISSION = 0;
+
+	/*** A constant indicating that a newsgroup is moderated. ***/
+	public static final int MODERATED_POSTING_PERMISSION = 1;
+
+	/*** A constant indicating that a newsgroup is public and unmoderated. ***/
+	public static final int PERMITTED_POSTING_PERMISSION = 2;
+
+	/***
+	 * A constant indicating that a newsgroup is closed for general posting.
+	 ***/
+	public static final int PROHIBITED_POSTING_PERMISSION = 3;
+
+	private String __newsgroup;
+	private long __estimatedArticleCount;
+	private long __firstArticle, __lastArticle;
+	private int __postingPermission;
+
+	void _setNewsgroup(String newsgroup) {
+		__newsgroup = newsgroup;
+	}
+
+	void _setArticleCount(long count) {
+		__estimatedArticleCount = count;
+	}
+
+	void _setFirstArticle(long first) {
+		__firstArticle = first;
+	}
+
+	void _setLastArticle(long last) {
+		__lastArticle = last;
+	}
+
+	void _setPostingPermission(int permission) {
+		__postingPermission = permission;
+	}
+
+	/***
+	 * Get the newsgroup name.
+	 * <p>
+	 * @return The name of the newsgroup.
+	 ***/
+	public String getNewsgroup() {
+		return __newsgroup;
+	}
+
+	/***
+	 * Get the estimated number of articles in the newsgroup.  The
+	 * accuracy of this value will depend on the server implementation.
+	 * <p>
+	 * @return The estimated number of articles in the newsgroup.
+	 ***/
+	public long getArticleCount() {
+		return __estimatedArticleCount;
+	}
+
+	/***
+	 * Get the number of the first article in the newsgroup.
+	 * <p>
+	 * @return The number of the first article in the newsgroup.
+	 ***/
+	public long getFirstArticle() {
+		return __firstArticle;
+	}
+
+	/***
+	 * Get the number of the last article in the newsgroup.
+	 * <p>
+	 * @return The number of the last article in the newsgroup.
+	 ***/
+	public long getLastArticle() {
+		return __lastArticle;
+	}
+
+	/***
+	 * Get the posting permission of the newsgroup.  This will be one of
+	 * the <code> POSTING_PERMISSION </code> constants.
+	 * <p>
+	 * @return The posting permission status of the newsgroup.
+	 ***/
+	public int getPostingPermission() {
+		return __postingPermission;
+	}
+
+	public String toString() {
+		StringBuilder buffer = new StringBuilder();
+		buffer.append(__newsgroup);
+		buffer.append(' ');
+		buffer.append(__lastArticle);
+		buffer.append(' ');
+		buffer.append(__firstArticle);
+		buffer.append(' ');
+		switch (__postingPermission) {
+		case 1:
+			buffer.append('m');
+			break;
+		case 2:
+			buffer.append('y');
+			break;
+		case 3:
+			buffer.append('n');
+			break;
+		}
+		return buffer.toString();
+	}
+
 }

Modified: jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/site/xdoc/changes.xml
                
URL: http://svn.apache.org/viewvc/jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/site/xdoc/changes.xml?view=diff&rev=523750&r1=523749&r2=523750
 ==============================================================================
--- jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/site/xdoc/changes.xml \
                (original)
+++ jakarta/commons/proper/net/branches/JDK_1_5_BRANCH/src/site/xdoc/changes.xml Thu \
Mar 29 08:37:00 2007 @@ -23,6 +23,9 @@
 
 	<body>
 		<release version="2.0" date="" description="Java 5.0 release">
+			<action dev="rwinston" type="update" issue="NET-155">
+				Change NNTPClient/NewsgroupInfo article counts from int to long
+			</action>
 			<action dev="rwinston" type="update" issue="NET-151">
 				Add support for explictly overriding destination hosts for passive sessions. \
Thanks to &lt;neil@JAMMConsulting.com&gt;  </action>



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


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic