[9] Review request: 8177566: FX user module gets IllegalAccessException from sun.reflect.misc.Trampoline
Kevin Rushforth
kevin.rushforth at oracle.com
Wed May 3 23:30:07 UTC 2017
JBS: https://bugs.openjdk.java.net/browse/JDK-8177566
Here is the updated webrev with (I hope) all comments addressed:
http://cr.openjdk.java.net/~kcr/8177566/webrev.01/complete-webrev/
For those who reviewed the earlier webrev, I have prepared delta webrevs.
* Delta webrev for the fix itself (just a slight change to the error
message, plus I hid the unused public methods in MethodUtil) :
http://cr.openjdk.java.net/~kcr/8177566/webrev.01/delta-fix-only.00/
* No changes in the tests
* Delta webrev for the doc changes:
http://cr.openjdk.java.net/~kcr/8177566/webrev.01/delta-doc-only.00/
* The sparse javadocs are also updated here:
http://cr.openjdk.java.net/~kcr/8177566/webrev.01/javadoc/
-- Kevin
Kevin Rushforth wrote:
> This review is being cross-posted to both openjfx-dev and jigsaw-dev.
>
> Please review the proposed fix for:
>
> https://bugs.openjdk.java.net/browse/JDK-8177566
> http://cr.openjdk.java.net/~kcr/8177566/webrev.00/complete-webrev/
>
> Details of the fix as well as notes to reviewers are in the bug report
> [1] (e.g., I've also generated separate webrevs for the fix itself,
> the doc changes, and the test changes).
>
> -- Kevin
>
> [1]
> https://bugs.openjdk.java.net/browse/JDK-8177566?focusedCommentId=14074243&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14074243
>
>
More information about the openjfx-dev
mailing list