Reviewed; it's good. The javadoc text doesn't need to predict the future; it just needs to document the present specification. So the sentence that begins "This constraint allows for the future possibility…" is not really necessary. It's certainly not normative.
— John On Apr 6, 2018, at 5:15 PM, Paul Sandoz <paul.san...@oracle.com> wrote: > > Hi, > > Please review this patch to constrain constant dynamic bootstrap methods to > methods whose first parameter type is MethodHandles.Lookup. > > http://cr.openjdk.java.net/~psandoz/jdk/JDK-8199875-condy-bsm-lookup/webrev/ > <http://cr.openjdk.java.net/~psandoz/jdk/JDK-8199875-condy-bsm-lookup/webrev/> > > We are conservatively diverging from invoke dynamic bootstrap method > invocation behaviour to possibly diverge further in the future and allow for > constant dynamic bootstrap methods that are invoked without the > lookup/name/type arguments. The change enables further divergence in a future > release without breaking compatibility. > > This would make it easier to use existing methods as bootstrap methods rather > than invoking via a level of indirection for explicit wrappers or using > ConstantBootstraps.invoke. The experience garnered from prototyping language > and low-level library features informs us this is useful. > > CSR is here: > > https://bugs.openjdk.java.net/browse/JDK-8201268 > <https://bugs.openjdk.java.net/browse/JDK-8201268> > > Thanks, > Paul.