<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