You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by cr...@apache.org on 2001/05/24 00:19:52 UTC
cvs commit: jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/util/xml XmlMapper.java
craigmcc 01/05/23 15:19:52
Modified: catalina/src/share/org/apache/catalina/util/xml
XmlMapper.java
Log:
Do not print double debug messages when processing a method setter that
declares zero arguments (and thus takes its method argument from the
nested body of the current tag).
PR: Patch #4 of the "8 Patches" email on 23 May 2001
Submitted by: Peter Costello <pe...@yahoo.com>
Revision Changes Path
1.6 +2 -2 jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/util/xml/XmlMapper.java
Index: XmlMapper.java
===================================================================
RCS file: /home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/util/xml/XmlMapper.java,v
retrieving revision 1.5
retrieving revision 1.6
diff -u -r1.5 -r1.6
--- XmlMapper.java 2000/11/24 22:36:25 1.5
+++ XmlMapper.java 2001/05/23 22:19:49 1.6
@@ -873,7 +873,7 @@
}
m.invoke( parent, realParam );
- if(ctx.getDebug() > 0 ) {
+ if ((ctx.getDebug() > 0) && (paramC > 0)) {
// debug
StringBuffer sb=new StringBuffer();
sb.append("" + parent.getClass().getName() + "." + mName + "( " );
@@ -882,7 +882,7 @@
sb.append(params[i]);
}
sb.append(")");
- if( ctx.getDebug() > 0 ) ctx.log(sb.toString());
+ ctx.log(sb.toString());
}
}
}