Changeset 17962 in josm

Timestamp:

2021-07-08T21:01:25+02:00 (3 years ago)

Author:

Don-vip

Message:

see #20522 - handle both language codes for Hebrew (iw before Java 17, he starting from Java 17)

See ​https://bugs.openjdk.java.net/browse/JDK-8263202

Location:

trunk

Files:

• src/org/openstreetmap/josm/tools/KeyboardUtils.java (modified) (1 diff)
• test/unit/org/openstreetmap/josm/tools/KeyboardUtilsTest.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/tools/KeyboardUtils.java

@@ -208 +208 @@
                 break;
             case "cs": // Czech
-            case "he": // Hebrew
+            case "he": // Hebrew starting from Java 17 - https://bugs.openjdk.java.net/browse/JDK-8263202
+            case "iw": // Hebrew before Java 17
                 // https://en.wikipedia.org/wiki/QWERTZ#Czech_(QWERTZ)
                 // https://en.wikipedia.org/wiki/Hebrew_keyboard

trunk/test/unit/org/openstreetmap/josm/tools/KeyboardUtilsTest.java

@@ -13 +13 @@
 import java.util.Map.Entry;
 
+import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
-import org.junit.jupiter.api.Test;
 import org.openstreetmap.josm.testutils.JOSMTestRules;
 
@@ -74 +74 @@
         testgetCharactersForKeyE00("de", deadCircumflex, deadCircumflex2);
         testgetCharactersForKeyE00("cs", ';');
-        testgetCharactersForKeyE00("he");
+        testgetCharactersForKeyE00("he", ';');
+        testgetCharactersForKeyE00("iw", ';');
         testgetCharactersForKeyE00("hu", '0');
         testgetCharactersForKeyE00("pl");