Mike, Doug,
On 28/05/2013 20:07, Mike Duigou wrote:
Hi Chris& Doug;
- I don't feel strongly about the removal of AbstractMap. I don't see this as
very likely to cause problems in real world code though there is probably some
test code somewhere that assigns CHM to an AbstractMap.
I don't feel strongly about this either, but I think it deserves
possibly its own bug number and consideration. I have removed it from
this review request, and will a file a new bug to track it.
- I am reluctant to deprecate contains(Object) here unless we deprecate it in
Hashtable as well. I recognize that this has been a source of errors
(https://issues.apache.org/bugzilla/show_bug.cgi?id=48755 for one example). Is
it time to deprecate it there as well?
Dito for this, removed from this request and should be revisited separately.
-Chris.
