You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ca...@apache.org on 2006/01/01 18:40:55 UTC
svn commit: r360502 -
/maven/plugins/trunk/maven-antrun-plugin/src/main/java/org/apache/maven/plugin/antrun/AntPropertyHelper.java
Author: carlos
Date: Sun Jan 1 09:36:46 2006
New Revision: 360502
URL: http://svn.apache.org/viewcvs?rev=360502&view=rev
Log:
Added log guards
Modified:
maven/plugins/trunk/maven-antrun-plugin/src/main/java/org/apache/maven/plugin/antrun/AntPropertyHelper.java
Modified: maven/plugins/trunk/maven-antrun-plugin/src/main/java/org/apache/maven/plugin/antrun/AntPropertyHelper.java
URL: http://svn.apache.org/viewcvs/maven/plugins/trunk/maven-antrun-plugin/src/main/java/org/apache/maven/plugin/antrun/AntPropertyHelper.java?rev=360502&r1=360501&r2=360502&view=diff
==============================================================================
--- maven/plugins/trunk/maven-antrun-plugin/src/main/java/org/apache/maven/plugin/antrun/AntPropertyHelper.java (original)
+++ maven/plugins/trunk/maven-antrun-plugin/src/main/java/org/apache/maven/plugin/antrun/AntPropertyHelper.java Sun Jan 1 09:36:46 2006
@@ -38,11 +38,12 @@
log = l;
}
- public synchronized Object getPropertyHook( String ns, String name,
- boolean user
- )
+ public synchronized Object getPropertyHook( String ns, String name, boolean user )
{
- log.debug( "getProperty(ns="+ns+", name="+name+", user="+user+")" );
+ if ( log.isDebugEnabled() )
+ {
+ log.debug( "getProperty(ns="+ns+", name="+name+", user="+user+")" );
+ }
Object val = null;
try
@@ -66,7 +67,10 @@
}
catch ( Exception e )
{
- log.warn( "Error evaluating expression '" + name + "'", e );
+ if ( log.isWarnEnabled() )
+ {
+ log.warn( "Error evaluating expression '" + name + "'", e );
+ }
e.printStackTrace();
}