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 2012/09/27 04:33:02 UTC

svn commit: r1390814 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/time/FastDateParser.java

Author: sebb
Date: Thu Sep 27 02:33:01 2012
New Revision: 1390814

URL: http://svn.apache.org/viewvc?rev=1390814&view=rev
Log:
Show where the parsing failed

Modified:
    commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/time/FastDateParser.java

Modified: commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/time/FastDateParser.java
URL: http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/time/FastDateParser.java?rev=1390814&r1=1390813&r2=1390814&view=diff
==============================================================================
--- commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/time/FastDateParser.java (original)
+++ commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/time/FastDateParser.java Thu Sep 27 02:33:01 2012
@@ -122,7 +122,8 @@ public class FastDateParser implements D
 
         Matcher patternMatcher= formatPattern.matcher(pattern);
         if(!patternMatcher.lookingAt()) {
-            throw new IllegalArgumentException("Invalid pattern");
+            throw new IllegalArgumentException(
+                    "Illegal pattern character '" + pattern.charAt(patternMatcher.regionStart()) + "'");
         }
 
         currentFormatField= patternMatcher.group();