<AWT Dev> RFR: 8270859: Post JEP 411 refactoring: client libs with maximum covering > 10K

[Kevin Rushforth]

On Fri, 16 Jul 2021 21:39:56 GMT, Vest <github.com+1671049+Vest at openjdk.org> wrote:

>> This is the last part of Post JEP 411 refactoring that makes `@SuppressWarnings("removal")` more fine grained. This fix deals with all client libs annotations that cover more than 10K bytes of code.
>
> src/java.desktop/share/classes/javax/swing/ImageIcon.java line 124:
> 
>> 122:                 // We don't care about component.
>> 123:                 // So don't prevent class initialisation.
>> 124:                 e.printStackTrace();
> 
> I am not the reviewer, but I am curious. If we do not care about the exception, is it a right way to just print it to the console as it is? If people use loggers, they won’t be able to capture this stack trace.
> thank you for your answer in advance.
> p.s. I know it is not your code, but isn’t it a suitable time to improve this part?

No. A refactoring change like this is the wrong time to make any unrelated changes. If such changes are a good idea, they should be done in a different bug with a separate pull request.

-------------

PR: https://git.openjdk.java.net/jdk/pull/4815