Author: olamy Date: Thu Apr 30 21:45:56 2009 New Revision: 770459 URL: http://svn.apache.org/viewvc?rev=770459&view=rev Log: add more information in case of date parsing exception.
Modified: maven/scm/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/util/AbstractConsumer.java Modified: maven/scm/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/util/AbstractConsumer.java URL: http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/util/AbstractConsumer.java?rev=770459&r1=770458&r2=770459&view=diff ============================================================================== --- maven/scm/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/util/AbstractConsumer.java (original) +++ maven/scm/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/util/AbstractConsumer.java Thu Apr 30 21:45:56 2009 @@ -77,9 +77,12 @@ { DateFormat format; + String patternUsed = null; + if ( StringUtils.isNotEmpty( userPattern ) ) { format = new SimpleDateFormat( userPattern ); + patternUsed = userPattern; } else { @@ -93,11 +96,14 @@ { format = new SimpleDateFormat( defaultPattern ); } + patternUsed = defaultPattern; } else { // Use the English short date pattern if no pattern is specified format = DateFormat.getDateInstance( DateFormat.SHORT, Locale.ENGLISH ); + + patternUsed = " DateFormat.SHORT "; } } @@ -109,7 +115,10 @@ { if ( getLogger() != null && getLogger().isErrorEnabled() ) { - getLogger().error( "ParseException: " + e.getMessage(), e ); + getLogger().error( + "skip ParseException: " + e.getMessage() + " during parsing date " + date + + " with pattern " + patternUsed + " with Locale " + + ( locale == null ? Locale.ENGLISH : locale ), e ); } return null;