<Swing Dev> [8] Review request for 7083457 Incomplete specification for javax/swing/text/DefaultCaret.html#setVisible(boolean)

sergey malenkov sergey.malenkov at oracle.com
Thu Aug 15 14:47:24 UTC 2013


Hi Alexander,

The fix is good, but there is no test in webrev.

Thanks,
SAM

On 15.08.2013 17:57, Alexander Scherbatiy wrote:
>
> Hello,
>
> Could you review the fix:
>   bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7083457
>   webrev: http://cr.openjdk.java.net/~alexsch/7083457/webrev.00
>
>   According to the javadoc the DefaultCaret.setVisible(boolean e) 
> should always update the getActive() state.
>
> Thanks,
> Alexandr.
>




More information about the swing-dev mailing list