Author: britter
Date: Thu Jan  1 16:23:15 2015
New Revision: 1648869

URL: http://svn.apache.org/r1648869
Log:
Assign logger only once

Modified:
    
commons/proper/validator/trunk/src/main/java/org/apache/commons/validator/util/ValidatorUtils.java

Modified: 
commons/proper/validator/trunk/src/main/java/org/apache/commons/validator/util/ValidatorUtils.java
URL: 
http://svn.apache.org/viewvc/commons/proper/validator/trunk/src/main/java/org/apache/commons/validator/util/ValidatorUtils.java?rev=1648869&r1=1648868&r2=1648869&view=diff
==============================================================================
--- 
commons/proper/validator/trunk/src/main/java/org/apache/commons/validator/util/ValidatorUtils.java
 (original)
+++ 
commons/proper/validator/trunk/src/main/java/org/apache/commons/validator/util/ValidatorUtils.java
 Thu Jan  1 16:23:15 2015
@@ -42,6 +42,8 @@ import org.apache.commons.validator.Var;
  */
 public class ValidatorUtils {
 
+    private static final Log LOG = LogFactory.getLog(ValidatorUtils.class);
+
     /**
      * <p>Replace part of a <code>String</code> with another value.</p>
      *
@@ -103,14 +105,11 @@ public class ValidatorUtils {
             value = PropertyUtils.getProperty(bean, property);
 
         } catch(IllegalAccessException e) {
-            Log log = LogFactory.getLog(ValidatorUtils.class);
-            log.error(e.getMessage(), e);
+            LOG.error(e.getMessage(), e);
         } catch(InvocationTargetException e) {
-            Log log = LogFactory.getLog(ValidatorUtils.class);
-            log.error(e.getMessage(), e);
+            LOG.error(e.getMessage(), e);
         } catch(NoSuchMethodException e) {
-            Log log = LogFactory.getLog(ValidatorUtils.class);
-            log.error(e.getMessage(), e);
+            LOG.error(e.getMessage(), e);
         }
 
         if (value == null) {


Reply via email to