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