On Tue, 20 May 2025 10:15:11 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:

> I see that `MenuItemTest` class exists in `bug4729669.java`:
> 
> https://github.com/openjdk/jdk/blob/e961b13cd68bc352b86af17c7e53df8537519beb/test/jdk/javax/swing/JMenuItem/bug4729669.java#L80
> 
> Is it similar to the one that you copied?
> 
> If that `MenuItemTest` class is identical, I suggest moving both tests into a 
> subdirectory and promoting `MenuItemTest` into a public class in that 
> subdirectory which is located in its own `.java` file.

No, these are no exactly the same, the doMenuItemTest(boolean isLeft) method in 
bug4729669.java has only one parameter, but the doMenuItemTest(boolean isLeft, 
String lafName, int frameY) method has three parameters in bug6197830.java.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/25319#issuecomment-2894001580

Reply via email to