I don't see the need for the extra parentheses in this particular case, but the change/fix looks good...

                        ...jim

On 06/30/2016 03:25 PM, Phil Race wrote:
Bug: https://bugs.openjdk.java.net/browse/JDK-8160693

Fix :-
  hg diff src/java.desktop/share/classes/sun/font/ScriptRunData.java
diff --git a/src/java.desktop/share/classes/sun/font/ScriptRunData.java
b/src/java.desktop/share/classes/sun/font/ScriptRunData.java
--- a/src/java.desktop/share/classes/sun/font/ScriptRunData.java
+++ b/src/java.desktop/share/classes/sun/font/ScriptRunData.java
@@ -44,7 +44,7 @@
          if (cp >= data[cache] && cp < data[cache+2]) {
              return data[cache+1];
          }
-        if (cp >= CHAR_START & cp < CHAR_LIMIT) {
+        if ((cp >= CHAR_START) && (cp < CHAR_LIMIT)) {
              int probe = dataPower;
              int index = 0;


-phil.

Reply via email to