You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by vi...@apache.org on 2013/07/01 13:18:54 UTC

svn commit: r1498384 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/catalina/valves/ErrorReportValve.java webapps/docs/changelog.xml

Author: violetagg
Date: Mon Jul  1 11:18:54 2013
New Revision: 1498384

URL: http://svn.apache.org/r1498384
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55159
Merged revision 1498368 from tomcat/trunk:
Small performance improvement
Patch provided by Adrian Nistor

Modified:
    tomcat/tc7.0.x/trunk/   (props changed)
    tomcat/tc7.0.x/trunk/java/org/apache/catalina/valves/ErrorReportValve.java
    tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc7.0.x/trunk/
------------------------------------------------------------------------------
  Merged /tomcat/trunk:r1498368

Modified: tomcat/tc7.0.x/trunk/java/org/apache/catalina/valves/ErrorReportValve.java
URL: http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/valves/ErrorReportValve.java?rev=1498384&r1=1498383&r2=1498384&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/java/org/apache/catalina/valves/ErrorReportValve.java (original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/catalina/valves/ErrorReportValve.java Mon Jul  1 11:18:54 2013
@@ -296,11 +296,12 @@ public class ErrorReportValve extends Va
         trace.append(t.toString()).append('\n');
         StackTraceElement[] elements = t.getStackTrace();
         int pos = elements.length;
-        for (int i = 0; i < elements.length; i++) {
+        for (int i = elements.length - 1; i >= 0; i--) {
             if ((elements[i].getClassName().startsWith
                  ("org.apache.catalina.core.ApplicationFilterChain"))
                 && (elements[i].getMethodName().equals("internalDoFilter"))) {
                 pos = i;
+                break;
             }
         }
         for (int i = 0; i < pos; i++) {

Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml?rev=1498384&r1=1498383&r2=1498384&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Mon Jul  1 11:18:54 2013
@@ -83,9 +83,9 @@
         fails to read a user&apos;s credentials. (markt)
       </fix>
       <fix>
-        <bug>55073</bug>, <bug>55108</bug>, <bug>55109</bug> &amp;
-        <bug>55110</bug>: Small performance improvements. Patches provided by
-        Adrian Nistor. (markt)
+        <bug>55073</bug>, <bug>55108</bug>, <bug>55109</bug>, <bug>55110</bug>
+        &amp; <bug>55159</bug>: Small performance improvements. Patches
+        provided by Adrian Nistor. (markt/violetagg)
       </fix>
       <add>
         <bug>55102</bug>: Add support for time to first byte in the



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org