https://issues.apache.org/bugzilla/show_bug.cgi?id=54241
Mark Thomas <ma...@apache.org> changed: What |Removed |Added ---------------------------------------------------------------------------- OS| |All --- Comment #1 from Mark Thomas <ma...@apache.org> --- 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: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org