Author: jim Date: Mon Aug 22 15:29:32 2011 New Revision: 1160303 URL: http://svn.apache.org/viewvc?rev=1160303&view=rev Log: works here
Modified: tomcat/tc5.5.x/trunk/STATUS.txt Modified: tomcat/tc5.5.x/trunk/STATUS.txt URL: http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/STATUS.txt?rev=1160303&r1=1160302&r2=1160303&view=diff ============================================================================== --- tomcat/tc5.5.x/trunk/STATUS.txt (original) +++ tomcat/tc5.5.x/trunk/STATUS.txt Mon Aug 22 15:29:32 2011 @@ -47,7 +47,7 @@ PATCHES PROPOSED TO BACKPORT: svn rm build/resources/tickyes.bmp svn rm build/resources/main.ico svn rm build/resources/uninst.ico - +1: markt + +1: markt, jim -1: * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=51550 @@ -66,10 +66,10 @@ PATCHES PROPOSED TO BACKPORT: The JkCoyoteHandler is a special case since it is a completely different implementation. However, JkCoyoteHandler does need to be changed to catch Throwable rather than Exception. I have - updated the proposed patch to do exactly that. + updated the proposed patch to do exactly that. -0: schultz: I tend to agree with Konstantin's assertion that the catch block is important. I understand the change to the status, but could you explain why the catch(Throwable) block has been removed? markt: See reply to kkolino above. - + -1: --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org