Author: pmouawad
Date: Mon Sep  3 20:09:10 2012
New Revision: 1380337

URL: http://svn.apache.org/viewvc?rev=1380337&view=rev
Log:
Add final to field
Add logger.isDebugEnabled

Modified:
    jmeter/trunk/src/components/org/apache/jmeter/assertions/XMLSchemaAssertion.java

Modified: jmeter/trunk/src/components/org/apache/jmeter/assertions/XMLSchemaAssertion.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/assertions/XMLSchemaAssertion.java?rev=1380337&r1=1380336&r2=1380337&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/assertions/XMLSchemaAssertion.java (original)
+++ jmeter/trunk/src/components/org/apache/jmeter/assertions/XMLSchemaAssertion.java Mon Sep  3 20:09:10 2012
@@ -163,7 +163,7 @@ public class XMLSchemaAssertion extends 
      * SAXErrorHandler class
      */
     private static class SAXErrorHandler implements ErrorHandler {
-        private AssertionResult result;
+        private final AssertionResult result;
 
         public SAXErrorHandler(AssertionResult result) {
             this.result = result;
@@ -175,7 +175,9 @@ public class XMLSchemaAssertion extends 
         public void error(SAXParseException exception) throws SAXParseException {
 
             String msg = "error: " + errorDetails(exception);
-            log.debug(msg);
+            if(log.isDebugEnabled()) {
+                log.debug(msg);
+            }
             result.setFailureMessage(msg);
             result.setError(true);
             throw exception;
@@ -188,7 +190,9 @@ public class XMLSchemaAssertion extends 
         public void fatalError(SAXParseException exception) throws SAXParseException {
 
             String msg = "fatal: " + errorDetails(exception);
-            log.debug(msg);
+            if(log.isDebugEnabled()) {
+                log.debug(msg);
+            }
             result.setFailureMessage(msg);
             result.setError(true);
             throw exception;
@@ -200,7 +204,9 @@ public class XMLSchemaAssertion extends 
         public void warning(SAXParseException exception) throws SAXParseException {
 
             String msg = "warning: " + errorDetails(exception);
-            log.debug(msg);
+            if(log.isDebugEnabled()) {
+                log.debug(msg);
+            }
             result.setFailureMessage(msg);
             // result.setError(true); // TODO is this the correct strategy?
             // throw exception; // allow assertion to pass