tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject [Bug 54241] NPE in BodyContentImpl
Date Tue, 04 Dec 2012 08:25:16 GMT

Mark Thomas <> changed:

           What    |Removed                     |Added
                 OS|                            |All

--- Comment #1 from Mark Thomas <> ---
I have a number of concerns with this proposed change:

1. Writers generally do throw NPEs when called with null.

2. I'm not convinced 35410 should have been fixed since toString() should never
return null (else all sorts of standard code patterns break).

3. The patch should ideally include a test case.

4. The proposed patch only addresses one possible NPE of many.

You are receiving this mail because:
You are the assignee for the bug.

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message