RFR: 8306871: Open source more AWT Drag & Drop tests [v2]

Damon Nguyen dnguyen at openjdk.org
Wed May 3 16:05:46 UTC 2023


On Tue, 2 May 2023 16:10:16 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:

>> Damon Nguyen has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix syntax. Edit comment
>
> test/jdk/java/awt/event/InputEvent/MouseModsTest.java line 145:
> 
>> 143: 
>> 144:     static int getKeycode(int ctrl) {
>> 145:         switch(ctrl) {
> 
> Suggestion:
> 
>         switch (ctrl) {

Fixed

> test/jdk/java/awt/event/InputEvent/MouseModsTest.java line 154:
> 
>> 152: 
>> 153:     static int getKeyModifier(int ctrl) {
>> 154:         switch(ctrl) {
> 
> Suggestion:
> 
>         switch (ctrl) {

Fixed

> test/jdk/java/awt/event/InputEvent/MouseModsTest.java line 163:
> 
>> 161: 
>> 162:     static int getMouseModifier(int btn) {
>> 163:         switch(btn) {
> 
> Suggestion:
> 
>         switch (btn) {

Fixed

> test/jdk/java/awt/event/KeyEvent/AltGraphModifier.java line 83:
> 
>> 81:                             if (modifierPress[i] == true) {
>> 82:                                 if ((kp.getModifiersEx() & inputMasks[i]) == inputMasks[i]) {
>> 83:                                 }  else {
> 
> Suggestion:
> 
>                                 } else {

Fixed

> test/jdk/java/awt/event/KeyEvent/KeyTyped/CancelKeyTyped.java line 114:
> 
>> 112: 
>> 113:             protected void printKey(KeyEvent evt) {
>> 114:                 switch(evt.getID()) {
> 
> Suggestion:
> 
>                 switch (evt.getID()) {

Fixed

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13725#discussion_r1183893293
PR Review Comment: https://git.openjdk.org/jdk/pull/13725#discussion_r1183893416
PR Review Comment: https://git.openjdk.org/jdk/pull/13725#discussion_r1183893511
PR Review Comment: https://git.openjdk.org/jdk/pull/13725#discussion_r1183893216
PR Review Comment: https://git.openjdk.org/jdk/pull/13725#discussion_r1183893133



More information about the client-libs-dev mailing list