svn commit: r509967 - /incubator/wicket/trunk/wicket/src/main/java/wicket/util/license/XmlLicenseHeaderHandler.java

[jbq]

Author: jbq
Date: Wed Feb 21 03:14:01 2007
New Revision: 509967

URL: http://svn.apache.org/viewvc?view=rev&rev=509967
Log:
WICKET-306 XML License Header Tests fail with Unicode BOM

Added logging to help me debug the issue

Modified:
    
incubator/wicket/trunk/wicket/src/main/java/wicket/util/license/XmlLicenseHeaderHandler.java

Modified: 
incubator/wicket/trunk/wicket/src/main/java/wicket/util/license/XmlLicenseHeaderHandler.java
URL: 
http://svn.apache.org/viewvc/incubator/wicket/trunk/wicket/src/main/java/wicket/util/license/XmlLicenseHeaderHandler.java?view=diff&rev=509967&r1=509966&r2=509967
==============================================================================
--- 
incubator/wicket/trunk/wicket/src/main/java/wicket/util/license/XmlLicenseHeaderHandler.java
 (original)
+++ 
incubator/wicket/trunk/wicket/src/main/java/wicket/util/license/XmlLicenseHeaderHandler.java
 Wed Feb 21 03:14:01 2007
@@ -20,6 +20,9 @@
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 import junit.framework.Assert;
 import wicket.util.diff.Diff;
 import wicket.util.diff.Revision;
@@ -28,7 +31,7 @@
 class XmlLicenseHeaderHandler extends AbstractLicenseHeaderHandler
 {
        private Pattern xmlHeader = 
Pattern.compile("^(\\<\\?xml[^"+LINE_ENDING+"]+?)"+LINE_ENDING+"(.*)$", 
Pattern.DOTALL | Pattern.MULTILINE);
-       
+       private static final Logger logger = 
LoggerFactory.getLogger(XmlLicenseHeaderHandler.class);
        /**
         * Construct.
         * 
@@ -56,9 +59,11 @@
                        if (header.startsWith("<?xml"))
                        {
                                header = 
header.substring(header.indexOf(LINE_ENDING) + LINE_ENDING.length());
+                               logger.debug("Header for " + file + ": " + 
header);
                        }
                        else
                        {
+                               logger.debug(file + " does not start with XML 
prolog");
                                // Then only take the first 16 lines
                                String[] headers = header.split(LINE_ENDING);
                                header = "";
@@ -73,6 +78,7 @@
                        }
 
                        revision = 
Diff.diff(getLicenseHeader().split(LINE_ENDING), header.split(LINE_ENDING));
+                       logger.debug("License diff for " + file + ": " + 
revision);
                }
                catch (Exception e)
                {