RFR: 8136583: Core libraries should use blessed modifier order
Paul Sandoz
paul.sandoz at oracle.com
Wed Sep 16 10:57:56 UTC 2015
On 16 Sep 2015, at 02:42, Martin Buchholz <martinrb at google.com> wrote:
> Hi, Chris and Paul,
> I'd like you to do a very boring code review.
>
> This change is entirely machine generated. (the script is more interesting)
> http://cr.openjdk.java.net/~martin/webrevs/openjdk9/blessed-modifier-order/blessed-modifier-order.patch
> https://bugs.openjdk.java.net/browse/JDK-8136583
AFAICT the script you used to produce the changes looks fine (a perl script embedded into a bash script :-) ).
After a certain point i cannot review the actual changes, my eye-balls give up :-) so i have to trust that it all compiles and tests run. Did you apply this patch internally and test?
Paul.
More information about the core-libs-dev
mailing list