On 16 Sep 2015, at 02:42, Martin Buchholz <[email protected]> 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.
