You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by yo...@apache.org on 2007/03/26 00:31:01 UTC

svn commit: r522363 - in /tomcat: container/tc5.5.x/webapps/docs/changelog.xml jasper/tc5.5.x/src/share/org/apache/jasper/JspC.java

Author: yoavs
Date: Sun Mar 25 15:30:59 2007
New Revision: 522363

URL: http://svn.apache.org/viewvc?view=rev&rev=522363
Log:
Bugzilla 41227: add a bit of DEBUG-level logging to JspC so users know what file is being compiled.

Modified:
    tomcat/container/tc5.5.x/webapps/docs/changelog.xml
    tomcat/jasper/tc5.5.x/src/share/org/apache/jasper/JspC.java

Modified: tomcat/container/tc5.5.x/webapps/docs/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/container/tc5.5.x/webapps/docs/changelog.xml?view=diff&rev=522363&r1=522362&r2=522363
==============================================================================
--- tomcat/container/tc5.5.x/webapps/docs/changelog.xml (original)
+++ tomcat/container/tc5.5.x/webapps/docs/changelog.xml Sun Mar 25 15:30:59 2007
@@ -67,6 +67,14 @@
       </fix>
     </changelog>
   </subsection>   
+  <subsection name="Jasper">
+    <changelog>
+      <fix>
+        <bug>41227</bug> Add a bit of DEBUG-level logging to JspC so users know
+          which file is being compiled. (yoavs)
+      </fix>
+    </changelog>
+  </subsection>
 </section>
 
 <section name="Tomcat 5.5.23 (fhanik)">

Modified: tomcat/jasper/tc5.5.x/src/share/org/apache/jasper/JspC.java
URL: http://svn.apache.org/viewvc/tomcat/jasper/tc5.5.x/src/share/org/apache/jasper/JspC.java?view=diff&rev=522363&r1=522362&r2=522363
==============================================================================
--- tomcat/jasper/tc5.5.x/src/share/org/apache/jasper/JspC.java (original)
+++ tomcat/jasper/tc5.5.x/src/share/org/apache/jasper/JspC.java Sun Mar 25 15:30:59 2007
@@ -813,6 +813,11 @@
     public void generateWebMapping( String file, JspCompilationContext clctxt )
         throws IOException
     {
+        if (log.isDebugEnabled()) {
+            log.debug("Generating web mapping for file " + file
+                      + " using compilation context " + clctxt);
+        }
+
         String className = clctxt.getServletClassName();
         String packageName = clctxt.getServletPackageName();
 
@@ -948,6 +953,10 @@
     private void processFile(String file)
         throws JasperException
     {
+        if (log.isDebugEnabled()) {
+            log.debug("Processing file: " + file);
+        }
+
         ClassLoader originalClassLoader = null;
 
         try {
@@ -989,6 +998,10 @@
             // Otherwise only generate .java, if .jsp file is newer than
             // the .java file
             if( clc.isOutDated(compile) ) {
+                if (log.isDebugEnabled()) {
+                    log.debug(jspUri + " is out dated, compiling...");
+                }
+
                 clc.compile(compile, true);
             }
 
@@ -1014,7 +1027,7 @@
             if(getFailOnError()) {
                 throw je;
             } else {
-                log.error(je.getMessage());
+                log.error(je.getMessage(), je);;
             }
 
         } catch (Exception e) {



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org