https://issues.apache.org/bugzilla/show_bug.cgi?id=44956
Raul Benito <[EMAIL PROTECTED]> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
--- Comment #3 from Raul Benito <[EMAIL PROTECTED]> 2008-05-14 14:11:29 PST ---
Good Analysis. I don't know why I forgot to remove add(null) in the contractor
that was the purpose of nullList, don't create and add a null to a list that
98% percent of the time will just have a null. And not to change the rest of
the code that depend on the null, as we don't have enough (any?) testcases that
check that behavior.
I have just add all the lines(including the unmodifiableList) even after
checking that the only add left checks before if the list is the null one. It
looks like a good way to detect the next error.
Thanks for detecting and for the report.
Regards
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
