<Swing Dev> [9] Review request for 8057770: api/javax_swing/JScrollPane/indexTGF.html#UpdateUI failed with MotifLookAndFeel on all platform

[Sergey Bylokhov]

Hi, Alexander.
The fix looks good.

On 09.09.2014 14:51, Alexander Zvegintsev wrote:
> Hello,
>
> please review the fix
> http://cr.openjdk.java.net/~azvegint/jdk/9/8057770/00/
> for the issue
> https://bugs.openjdk.java.net/browse/JDK-8057770
>
> This  issue
> MotifScrollPaneUI does not override uninstallListeners() method from 
> BasicScrollPaneUI
>
> It has following signature
> protected void uninstallListeners(JScrollPane scrollPane)
> but it should be
> protected void uninstallListeners(JComponent c)
>
> Hence listeners will never be deleted. This type of issues can be 
> avoided by using @Override annotation.
> Same issue fixed in XAWTScrollPaneUI.
>
>
>
>


-- 
Best regards, Sergey.