On Mon, 24 Apr 2023 17:39:17 GMT, Tejesh R <[email protected]> wrote:

> Open source few AWT ScrollPane related tests.

I thought the preferred layout for tests is to use a flat layout without 
creating *additional folder* for each test.

So, `test/jdk/java/awt/ScrollPane/ComponentScrollTest/ComponentScrollTest.java` 
should be under `ScrollPane`: 
`test/jdk/java/awt/ScrollPane/ComponentScrollTest.java`.

Similarly, 
`test/jdk/java/awt/ScrollPane/ScrollPaneInsets/ScrollPaneExtraScrollBar.java` 
goes to `test/jdk/java/awt/ScrollPane/ScrollPaneExtraScrollBar.java`.

The test `test/jdk/java/awt/ScrollPane/RemoveChild` consists of two .java 
source files. Can the Semaphore class be moved into `ScrollPaneRemoveAdd.java` 
as a static nested class?

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

PR Review: https://git.openjdk.org/jdk/pull/13621#pullrequestreview-1398613298

Reply via email to