On Tue, 22 Mar 2022 11:30:13 GMT, Alexey Ivanov <[email protected]> wrote:
>> Tejesh R has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Updated based on review comments
>
> src/java.desktop/share/classes/javax/swing/ImageIcon.java line 341:
>
>> 339: {
>> 340: loadStatus = MediaTracker.ABORTED;
>> 341: }
>
> Did you compile this code?
> I can't see `interrupted` variable being declared.
>
> Please follow Java Coding Style: a space between `if` and the opening
> parenthesis, the opening brace is on the same line rather than on its own
> line. The code that [I posted as a
> sample](https://github.com/openjdk/jdk/pull/7754#discussion_r831096445)
> follows the Java Style.
Previous code I compiled and tested, the one I committed was not compiled and
tested yet, had to commit coz I'm restoring my local repo, after restoring will
make the changes and test locally and with mach5......
-------------
PR: https://git.openjdk.java.net/jdk/pull/7754
