<div dir="auto"><div>Hello, Philip!<div dir="auto"><br></div><div dir="auto">No, I didn't submit a job. Please do it. Thanks!</div><div dir="auto"><br></div><div dir="auto">With best regards, </div><div dir="auto">Tagir Valeev.</div><br><br><div class="gmail_quote"><div dir="ltr">пн, 1 окт. 2018 г., 23:45 Philip Race <<a href="mailto:philip.race@oracle.com">philip.race@oracle.com</a>>:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
On 9/30/18, 8:29 PM, Tagir Valeev wrote:<br>
> Hello!<br>
><br>
> Please review JDK-8211300 [1] this change: [2]. Also it needs a<br>
> sponsor as I have only JDK author status in OpenJDK Census [3].<br>
<br>
Adding awt + swing.<br>
I can sponsor this. Have you submitted a build via <br>
<a href="http://hg.openjdk.java.net/jdk/submit" rel="noreferrer noreferrer" target="_blank">hg.openjdk.java.net/jdk/submit</a> ?<br>
I can submit a job if you have not.<br>
I have also made a few updates to the bug report.<br>
<br>
-phil.<br>
><br>
> The discussion in core-libs-dev [4] shows that it's desired to get rid<br>
> of old style array declarations like `int array[]` and massively<br>
> convert them to `int[] array`. I volunteered to work on this. As Alan<br>
> Bateman noted [5], java.desktop module is pushed to separate repo, so<br>
> it would be better to process it separately, thus I'd like to start<br>
> with it.<br>
><br>
> The changeset was created in the following steps:<br>
> * Import JDK sources to IntelliJ IDEA<br>
> * Mark java.desktop/aix/classes, java.desktop/macosx/classes,<br>
> java.desktop/unix/classes, java.desktop/solaris/classes,<br>
> java.desktop/windows/classes and java.desktop/share/classes as source<br>
> roots<br>
> * Disable automatic formatting on the whole project<br>
> * Run the inspection "C-style array declaration"<br>
> * Apply the quick-fix massively<br>
> * Perform regex replace over changed files only:<br>
> Search: Copyright \(c\) (\d+), (\d+), Oracle and/or its affiliates.<br>
> All rights reserved.<br>
> Replace: Copyright \(c\) $1, 2018, Oracle and/or its affiliates. All<br>
> rights reserved.<br>
> * Perform regex replace over changed files only:<br>
> Search: Copyright \(c\) (\d+), Oracle and/or its affiliates. All<br>
> rights reserved.<br>
> Replace: Copyright \(c\) $1, 2018, Oracle and/or its affiliates. All<br>
> rights reserved.<br>
> * It appeared that compound declarations like `byte r[], g[], b[];`<br>
> are converted to `byte[] r;byte[] g; byte[] b;` which does not look<br>
> nice while compilable. I found three such cases via simple regexp<br>
> search in BMPImageReader#658, BMPImageWriter#325 and<br>
> AreaAveragingScaleFilter#66 and fixed them manually.<br>
><br>
> In total 339 files were changed with 1335 lines of array declaration<br>
> updates and 310 lines of copyright updates. I reviewed the generated<br>
> patch by eyes, but only partially, because it's too big. Also I<br>
> performed some kind of simple sanity check using regexps:<br>
><br>
> $ grep '^+[^+]' jdk.patch | grep -v 'Oracle and/or its affiliates. All<br>
> rights reserved'|grep -v '\[\]'|wc<br>
>        0       0       0<br>
> (check that every updated line contains either copyright message or [])<br>
><br>
> With best regards,<br>
> Tagir Valeev.<br>
><br>
> [1] <a href="https://bugs.openjdk.java.net/browse/JDK-8211300" rel="noreferrer noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8211300</a><br>
> [2] <a href="http://cr.openjdk.java.net/~tvaleev/webrev/8211300/r1/" rel="noreferrer noreferrer" target="_blank">http://cr.openjdk.java.net/~tvaleev/webrev/8211300/r1/</a><br>
> [3] <a href="http://openjdk.java.net/census#tvaleev" rel="noreferrer noreferrer" target="_blank">http://openjdk.java.net/census#tvaleev</a><br>
> [4] <a href="http://mail.openjdk.java.net/pipermail/core-libs-dev/2018-September/055390.html" rel="noreferrer noreferrer" target="_blank">http://mail.openjdk.java.net/pipermail/core-libs-dev/2018-September/055390.html</a><br>
> [5] <a href="http://mail.openjdk.java.net/pipermail/core-libs-dev/2018-September/055759.html" rel="noreferrer noreferrer" target="_blank">http://mail.openjdk.java.net/pipermail/core-libs-dev/2018-September/055759.html</a><br>
</blockquote></div></div></div>