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

Reply via email to