RFR: 8001647: In-place methods on Collection/List
akhil.arora at oracle.com
Sat Apr 20 01:39:19 UTC 2013
On 04/19/2013 04:15 AM, Alan Bateman wrote:
> On 18/04/2013 19:49, Akhil Arora wrote:
>> Looks like the stars are aligning on getting on this into TL... the
>> refreshed webrev is -
> A minor comment on Collection.removeIf is "that satisifies the given
> predicate" might be better than "which matches the provided predicate".
> Also for completeness, you could say "RuntimeExceptions and Errors
> thrown by the predicate are propagated ...".
> In List.replaceAll then @throws NullPointerException is listed twice,
> which is okay, but might be better to combine them. A typo in the second
> NPE description "if the an element".
combined and fixed NPE
> In the implementation then the only thing that puzzled me is checking
> the modification count in legacy Vector, that seems unnecessary.
> I see Mike has looked through the tests in detail. One additional
> comment is just location and trying to keep it consistent with the
> current organization if possible.
moved the tests to java/util/Collection
More information about the core-libs-dev