<Swing Dev> Integrated: 8263496: MetalHighContrastTheme.getControlHighlight cleanup

Prasanta Sadhukhan psadhukhan at openjdk.java.net
Thu Apr 1 07:38:32 UTC 2021


On Tue, 23 Mar 2021 10:26:42 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

> SonarCloud reports the potential issue with MetalHighContrastTheme.getControlHighlight where `controlHighlight ` field is not used and the getControlHighlight() uses secondary field.
> public ColorUIResource getControlHighlight() {
>         // This was super.getSecondary3();
>         return secondary2;
>     } 
> Removed the unused field.

This pull request has now been integrated.

Changeset: 0696fd0e
Author:    Prasanta Sadhukhan <psadhukhan at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/0696fd0e
Stats:     2 lines in 1 file changed: 0 ins; 2 del; 0 mod

8263496: MetalHighContrastTheme.getControlHighlight cleanup

Reviewed-by: azvegint

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

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


More information about the swing-dev mailing list