RFR: JDK-8328012 : Convert InputMethod (/java/awt/im) applet tests to main [v4]

Harshitha Onkar honkar at openjdk.org
Sat Mar 16 22:35:19 UTC 2024


On Thu, 14 Mar 2024 04:43:48 GMT, Tejesh R <tr at openjdk.org> wrote:

>> Harshitha Onkar has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   instruction updated
>
> test/jdk/java/awt/im/PinyinIMCommaTest.java line 44:
> 
>> 42: 
>> 43: public class PinyinIMCommaTest {
>> 44:     private static final String INSTRUCTIONS = """
> 
> `INSTRUCTIONS ` can be moved inside main  (Other test also)

Since it is okay to have it either as static var or within main, I'm going to leave it as it is now.

> test/jdk/java/awt/im/PinyinIMCommaTest.java line 82:
> 
>> 80:         JTextField input = new JTextField(20);
>> 81:         panel.add(new JLabel("Text field:"));
>> 82:         panel.add(input);
> 
> can add JTextField also this way `panel.add(new JTextField(20));`?

Updated.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18251#discussion_r1526685355
PR Review Comment: https://git.openjdk.org/jdk/pull/18251#discussion_r1526685550


More information about the client-libs-dev mailing list