RFR: 8023339 & 8023340 : (xs) Refine throws UOE/NPE Conditions
Mike Duigou
mike.duigou at oracle.com
Fri Sep 27 20:30:56 UTC 2013
Corrected before push.
Mike
On Sep 20 2013, at 12:11 , Paul Sandoz wrote:
> Hi Mike,
>
> You made changes to the documentation of List.replaceAll which does not perform matching:
>
> 398 * @param operator the operator to apply to each element
> 399 * @throws UnsupportedOperationException if this list is unmodifiable.
> 400 * Implementations may throw this exception if a matching element
> 401 * cannot be replaced or if, in general, modification is not
> 402 * supported
> 403 * @throws NullPointerException if the specified operator is null or
> 404 * if the operator result is a null value and this list does
> 405 * not permit null elements
> 406 * (<a href="Collection.html#optional-restrictions">optional</a>)
> 407 * @since 1.8
> 408 */
> 409 default void replaceAll(UnaryOperator<E> operator) {
>
> * @throws UnsupportedOperationException if this list is unmodifiable.
> * Implementations may throw this exception if an element
> * cannot be replaced or if, in general, modification is not
> * supported
>
> ?
>
> Paul.
>
> On Sep 17, 2013, at 3:35 PM, Mike Duigou <mike.duigou at oracle.com> wrote:
>
>> Hello all;
>>
>> Another, hopefully final, attempt at refining the @throws UOE javadoc for Collections.removeIf and the @throws UOE and NPE javadoc for List.replaceAll().
>>
>> This cycle adopts the verbiage suggested by Paul Sandoz in the last round of 8023339.
>>
>> Both changesets are combined into a single webrev:
>>
>> http://cr.openjdk.java.net/~mduigou/JDK-8023340/1/webrev/
>>
>> Thanks,
>>
>> Mike
>
More information about the core-libs-dev
mailing list