You are viewing a plain text version of this content. The canonical link for it is here.
Posted to fop-commits@xmlgraphics.apache.org by vh...@apache.org on 2007/02/28 09:45:27 UTC

svn commit: r512661 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/layoutmgr/table/TableStepper.java

Author: vhennebert
Date: Wed Feb 28 00:45:26 2007
New Revision: 512661

URL: http://svn.apache.org/viewvc?view=rev&rev=512661
Log:
Add code guards around log statements for better performance when they are disabled

Modified:
    xmlgraphics/fop/trunk/src/java/org/apache/fop/layoutmgr/table/TableStepper.java

Modified: xmlgraphics/fop/trunk/src/java/org/apache/fop/layoutmgr/table/TableStepper.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/trunk/src/java/org/apache/fop/layoutmgr/table/TableStepper.java?view=diff&rev=512661&r1=512660&r2=512661
==============================================================================
--- xmlgraphics/fop/trunk/src/java/org/apache/fop/layoutmgr/table/TableStepper.java (original)
+++ xmlgraphics/fop/trunk/src/java/org/apache/fop/layoutmgr/table/TableStepper.java Wed Feb 28 00:45:26 2007
@@ -154,7 +154,9 @@
         for (int i = 0; i < rowGroup.length; i++) {
             totalHeight += rowGroup[i].getHeight().opt;
         }
-        log.debug("totalHeight=" + totalHeight);
+        if (log.isDebugEnabled()) {
+            log.debug("totalHeight=" + totalHeight);
+        }
     }
     
     private int getMaxRemainingHeight() {
@@ -238,7 +240,9 @@
                 borderBefore[column] = pgu.getBorders().getBorderBeforeWidth(false);
             } else {
                 borderBefore[column] = pgu.getBorders().getBorderBeforeWidth(false) / 2;
-                log.trace("border before for column " + column + ": " + borderBefore[column]);
+                if (log.isTraceEnabled()) {
+                    log.trace("border before for column " + column + ": " + borderBefore[column]);
+                }
             }
             paddingBefore[column] = pgu.getBorders().getPaddingBefore(false, pgu.getCellLM());
             paddingAfter[column] = pgu.getBorders().getPaddingAfter(false, pgu.getCellLM());



---------------------------------------------------------------------
To unsubscribe, e-mail: fop-commits-unsubscribe@xmlgraphics.apache.org
For additional commands, e-mail: fop-commits-help@xmlgraphics.apache.org