https://issues.apache.org/bugzilla/show_bug.cgi?id=47216
Mark Thomas <ma...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |WONTFIX
--- Comment #3 from Mark Thomas <ma...@apache.org> 2009-05-25 04:00:45 PST ---
The answer is in the svn history. See r298828 (
https://svn.apache.org/viewcvs.cgi?view=rev&rev=298828 ) and r299051 (
https://svn.apache.org/viewcvs.cgi?view=rev&rev=299051 ).
It may be the case that later changes have removed the need for some of these
null tests. A cursory review suggests that a few of the tests could be removed
but the risk of introducing an NPE for an unusual code path outweighs the
minimal benefit of removing these tests.
I have changed the scope of parseHost() to protected for trunk. It will not be
backported as it breaks the API.
Since the (socket != null) was the main thrust of this report, I am marking it
as WONTFIX.
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- 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
