> On 22 Feb 2016, at 12:40, Michael Haupt <[email protected]> wrote:
>
> Dear all,
>
> please review this (doc-only) fix.
> Bug: https://bugs.openjdk.java.net/browse/JDK-8150360
> Webrev: http://cr.openjdk.java.net/~mhaupt/8150360/webrev.00
>
> The fix addresses one contradiction in the API documentation of
> MethodHandles.tryFinally(), and adds the following note to the API
> documentation of the method handle combinators given below:
>
> + * <p>
> + * <em>Note:</em> The resulting adapter is never a {@linkplain
> MethodHandle#asVarargsCollector
> + * variable-arity method handle}, even if the original target method
> handle was.
>
> * MethodHandle.asCollector(), .bindTo()
> * MethodHandles.collectArguments(), .dropArguments(), .filterArguments(),
> .filterReturnValue(), .foldArguments(), .permuteArguments()
>
Looks good. Minor thing, no need for another webrev round.
MethodHandles
—
3849 * The {@code target} and {@code cleanup} handles must have the same
corresponding argument and return types, except
3850 * that the {@code cleanup} handle may omit trailing arguments. Also,
the {@code cleanup} must have one or two extra
3851 * leading parameters:<ul>
Either “Also, the {@code cleanup} handle must...” or “Also, {@code cleanup}
must…” ?
Paul.
