<div dir="ltr">Looks good to me!</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 18, 2015 at 2:37 PM, Phil Race <span dir="ltr"><<a href="mailto:philip.race@oracle.com" target="_blank">philip.race@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><a href="http://cr.openjdk.java.net/~prr/8136783/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~prr/8136783/</a><br>
<a href="https://bugs.openjdk.java.net/browse/JDK-8136783" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8136783</a><br>
<br>
The idea here is to have our sources follow the<br>
recommended order fom the JLS which is further documented here<br>
<a href="https://docs.oracle.com/javase/8/docs/api/java/lang/reflect/Modifier.html#toString-int-" rel="noreferrer" target="_blank">https://docs.oracle.com/javase/8/docs/api/java/lang/reflect/Modifier.html#toString-int-</a><br>
<br>
For background this was applied to core-libs here :<br>
<a href="http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-September/035217.html" rel="noreferrer" target="_blank">http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-September/035217.html</a><br>
<br>
The script was checked in here :<br>
<a href="http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-September/035246.html" rel="noreferrer" target="_blank">http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-September/035246.html</a><br>
<br>
It was proposed to run it on the rest of the JDK here :<br>
<a href="http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-September/035273.html" rel="noreferrer" target="_blank">http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-September/035273.html</a><br>
<br>
What I am updating here is all the files under src/java.desktop which<br>
is a decent-sized chunk.: 324 files. I recommend looking at the jdk.patch file<br>
<br>
I have checked the changes myself visually but also used JPRT to<br>
make sure that no introduced syntax error causes a build failure.<br>
<br>
Separately 2 minor changes to closed source files have been sent for internal review.<span class="HOEnZb"><font color="#888888"><br>
<br>
-phil.<br>
</font></span></blockquote></div><br></div>