> On 17 Oct 2016, at 15:01, Stuart Marks <stuart.ma...@oracle.com> wrote:
> 
> Hi Paul,
> 
> I took a look at the jdk changes. They look good to me.
> 
> One section of code gave me pause, which is the throw of ClassCastException 
> at 339 of CallSite.java, and the throw of the exception returned from 
> wrongTargetType() at 344 of CallSite.java. This appears odd given the 
> "rethrow any Error and wrap anything else in BSME" rule. But these throws are 
> caught by the catch of Throwable below, which does the wrap and throw of BSME.
> 
> This arrangement was already present in the code.
> 
> Usually I wrinkle my nose at a throw that's caught by a catch clause later 
> on, but in this case it's not obvious what would be better. Maybe a comment 
> is warranted?

In addition to the

  // See the "Linking Exceptions" section for the invokedynamic
  // instruction in JVMS 6.5.

I can add something like:

  // Throws a runtime exception defining the cause that is then later wrapped 
in BootstrapMethodError

?

Thanks,
Paul.

Reply via email to