You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by kk...@apache.org on 2012/02/17 01:49:08 UTC
svn commit: r1245274 -
/tomcat/trunk/java/org/apache/catalina/mbeans/MBeanDumper.java
Author: kkolinko
Date: Fri Feb 17 00:49:08 2012
New Revision: 1245274
URL: http://svn.apache.org/viewvc?rev=1245274&view=rev
Log:
Skip debug call if debugging is not enabled.
Modified:
tomcat/trunk/java/org/apache/catalina/mbeans/MBeanDumper.java
Modified: tomcat/trunk/java/org/apache/catalina/mbeans/MBeanDumper.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/mbeans/MBeanDumper.java?rev=1245274&r1=1245273&r2=1245274&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/mbeans/MBeanDumper.java (original)
+++ tomcat/trunk/java/org/apache/catalina/mbeans/MBeanDumper.java Fri Feb 17 00:49:08 2012
@@ -83,11 +83,15 @@ public class MBeanDumper {
} catch (JMRuntimeException rme) {
Throwable cause = rme.getCause();
if (cause instanceof UnsupportedOperationException) {
- log.debug("Error getting attribute " + oname +
- " " + attName, rme);
+ if (log.isDebugEnabled()) {
+ log.debug("Error getting attribute " + oname
+ + " " + attName, rme);
+ }
} else if (cause instanceof NullPointerException) {
- log.debug("Error getting attribute " + oname +
- " " + attName, rme);
+ if (log.isDebugEnabled()) {
+ log.debug("Error getting attribute " + oname
+ + " " + attName, rme);
+ }
} else {
log.error("Error getting attribute " + oname +
" " + attName, rme);
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org