On Fri, 6 Jun 2025 09:36:09 GMT, Tejesh R <t...@openjdk.org> wrote:

>> Jeremy Wood has updated the pull request incrementally with two additional 
>> commits since the last revision:
>> 
>>  - 8354646: change Error to RuntimeException
>>    
>>    This is in response to:
>>    https://github.com/openjdk/jdk/pull/25443#discussion_r2131681790
>>  - 8354646: removing catching RuntimeException
>>    
>>    This is in response to:
>>    https://github.com/openjdk/jdk/pull/25443#discussion_r2131680454
>
> test/jdk/javax/swing/plaf/basic/BasicPasswordFieldUI/PasswordSelectionWordTest.java
>  line 45:
> 
>> 43: public class PasswordSelectionWordTest {
>> 44:     public static void main(String[] args) throws Exception {
>> 45:         for (UIManager.LookAndFeelInfo laf :
> 
> Now since the test is for all the look and feel, should it be inside 
> `test/jdk/javax/swing/JPasswordField` folder?
> Other than this it looks good to me.

Okay; this is moved.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25443#discussion_r2135102051

Reply via email to