RFR: 8316056: Open source several Swing JTree tests [v2]
Damon Nguyen
dnguyen at openjdk.org
Mon Sep 18 19:09:48 UTC 2023
On Mon, 18 Sep 2023 14:10:01 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>> Damon Nguyen has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Review comments changes
>
> test/jdk/javax/swing/JTree/bug4213868.java line 54:
>
>> 52: JTree parent = createTree();
>> 53: AccessibleContext c = parent.getAccessibleContext()
>> 54: .getAccessibleChild(0).getAccessibleContext();
>
> Suggestion:
>
> AccessibleContext c = parent.getAccessibleContext()
> .getAccessibleChild(0)
> .getAccessibleContext();
Added the newlines and alignments, thanks
> test/jdk/javax/swing/JTree/bug4213868.java line 56:
>
>> 54: .getAccessibleChild(0).getAccessibleContext();
>> 55: if (c.getAccessibleChild(1).getAccessibleContext()
>> 56: .getAccessibleIndexInParent() != 1) {
>
> Suggestion:
>
> if (c.getAccessibleChild(1)
> .getAccessibleContext()
> .getAccessibleIndexInParent() != 1) {
>
> However, it doesn't look as good as the above.
Added the newlines and alignments, thanks
> test/jdk/javax/swing/JTree/bug4224491.java line 43:
>
>> 41:
>> 42: public static void main(String[] args) throws InterruptedException,
>> 43: InvocationTargetException {
>
> Suggestion:
>
> public static void main(String[] args) throws Exception {
Updated where applicable
> test/jdk/javax/swing/JTree/bug4237370.java line 56:
>
>> 54: }
>> 55:
>> 56: public void treeExpanded (TreeExpansionEvent e) {
>
> Suggestion:
>
> public void treeExpanded(TreeExpansionEvent e) {
Fixed
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15756#discussion_r1329163276
PR Review Comment: https://git.openjdk.org/jdk/pull/15756#discussion_r1329163175
PR Review Comment: https://git.openjdk.org/jdk/pull/15756#discussion_r1329162846
PR Review Comment: https://git.openjdk.org/jdk/pull/15756#discussion_r1329162574
More information about the client-libs-dev
mailing list