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;


Reply via email to