You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm-commits@maven.apache.org by ol...@apache.org on 2009/04/30 23:45:56 UTC
svn commit: r770459 -
/maven/scm/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/util/AbstractConsumer.java
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;