https://bz.apache.org/bugzilla/show_bug.cgi?id=61311
Mark Thomas <ma...@apache.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |INVALID --- Comment #2 from Mark Thomas <ma...@apache.org> --- It looks weird but it is correct. The implementation of setParseFailedReason() only updates the reason if no reason is set. A more efficient finally block is almost certainly possible but the current code is intended to be robust with respect to changes in the rest of the method. I'll add a comment for future reference. -- You are receiving this mail because: You are the assignee for the bug. --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org