On Mon, 6 Jun 2022 06:08:38 GMT, Tejesh R <t...@openjdk.org> wrote:
>> Added test for checking setMargin() of JRadioButton.
>
> Tejesh R has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Updated based on review comments
Changes requested by aivanov (Reviewer).
test/jdk/javax/swing/JRadioButton/bug4380543.java line 58:
> 56: 2. Rendering depends on OS and supported Look and Feels.
> 57: Verify only with those L&F where margins are visible.
> 58: 3. If Left inset(margin) appear too small, press Fail,
Suggestion:
3. If Left inset(margin) appears too small, press Fail,
test/jdk/javax/swing/JRadioButton/bug4380543.java line 72:
> 70: //Adding the Test Frame to handle dispose
> 71: passFailJFrame.addTestFrame(testObj);
> 72: passFailJFrame.positionTestFrame(testObj,
> PassFailJFrame.Position.HORIZONTAL);
This was correct: you're calling static methods; such methods should not be
called _via an instance variable_ but via the class name. Alternatively, you
can static-import these methods.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8721
