Hi Roman,
could you please add a comment to these methods that says that the NPEs are
supposed to be thrown implicitly. Otherwise I find it a bit odd that the javadoc
mentions the exceptions explicitly.

cya
Robert

Roman Kennke wrote:
> This removes 2 explicit null checks in Vector. The Mauve test that I'll
> commit right after this shows that the RI allows null arguments when the
> Vector is empty. In the other case we throw an NPE implicitly anyway.
> 
> 2006-08-15  Roman Kennke  <[EMAIL PROTECTED]>
> 
>     * java/util/Vector.java
>     (removeAll): Don't explicitly null-check here. The RI allows
>     null arguments when Vector is empty. In other cases we
>     implicitly throw an NPE.
>     (retainAll): Don't explicitly null-check here. The RI allows
>     null arguments when Vector is empty. In other cases we
>     implicitly throw an NPE.
> 
> /Roman
> 

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to