You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2011/04/07 23:30:47 UTC

svn commit: r1090027 - /commons/proper/net/trunk/src/main/java/org/apache/commons/net/ftp/parser/FTPTimestampParserImpl.java

Author: sebb
Date: Thu Apr  7 21:30:47 2011
New Revision: 1090027

URL: http://svn.apache.org/viewvc?rev=1090027&view=rev
Log:
ParserException should be passed the errorIndex

Modified:
    commons/proper/net/trunk/src/main/java/org/apache/commons/net/ftp/parser/FTPTimestampParserImpl.java

Modified: commons/proper/net/trunk/src/main/java/org/apache/commons/net/ftp/parser/FTPTimestampParserImpl.java
URL: http://svn.apache.org/viewvc/commons/proper/net/trunk/src/main/java/org/apache/commons/net/ftp/parser/FTPTimestampParserImpl.java?rev=1090027&r1=1090026&r2=1090027&view=diff
==============================================================================
--- commons/proper/net/trunk/src/main/java/org/apache/commons/net/ftp/parser/FTPTimestampParserImpl.java (original)
+++ commons/proper/net/trunk/src/main/java/org/apache/commons/net/ftp/parser/FTPTimestampParserImpl.java Thu Apr  7 21:30:47 2011
@@ -95,7 +95,7 @@ public class FTPTimestampParserImpl impl
         Calendar now = (Calendar) serverTime.clone();// Copy this, because we may change it
         now.setTimeZone(this.getServerTimeZone());
         Calendar working = (Calendar) now.clone();
-        working.setTimeZone(getServerTimeZone());
+        working.setTimeZone(getServerTimeZone()); // is this needed?
         ParsePosition pp = new ParsePosition(0);
 
         Date parsed = null;
@@ -149,7 +149,7 @@ public class FTPTimestampParserImpl impl
                 } else {
                     throw new ParseException(
                             "Timestamp could not be parsed with older or recent DateFormat",
-                            pp.getIndex());
+                            pp.getErrorIndex());
                 }
             }
         }