commit d304d60e4d49df14ed85d6646680085f27bafbf2
author Behdad Esfahbod <behdad@behdad.org> Wed Aug 21 12:30:22 2019 -0700
committer Behdad Esfahbod <behdad@behdad.org> Wed Aug 21 12:30:22 2019 -0700
tree dc94e1e855bd101cc30b7b23cb56b9b61e832e33
parent 2a3d4987a75fb2cd51ccf4c1d08baba383ceda7b

[ot-font] Prefer symbol cmap subtable if found

Fixes https://github.com/harfbuzz/harfbuzz/issues/1918

Hopefully doesn't break anyone...

src/hb-ot-cmap-table.hh

diff --git a/src/hb-ot-cmap-table.hh b/src/hb-ot-cmap-table.hh
index 746e87f..23ea465 100644
--- a/src/hb-ot-cmap-table.hh
+++ b/src/hb-ot-cmap-table.hh
@@ -993,6 +993,15 @@
 
     const CmapSubtable *subtable;
 
+    /* Symbol subtable.
+     * Prefer symbol if available.
+     * https://github.com/harfbuzz/harfbuzz/issues/1918 */
+    if ((subtable = this->find_subtable (3, 0)))
+    {
+      if (symbol) *symbol = true;
+      return subtable;
+    }
+
     /* 32-bit subtables. */
     if ((subtable = this->find_subtable (3, 10))) return subtable;
     if ((subtable = this->find_subtable (0, 6))) return subtable;
@@ -1005,13 +1014,6 @@
     if ((subtable = this->find_subtable (0, 1))) return subtable;
     if ((subtable = this->find_subtable (0, 0))) return subtable;
 
-    /* Symbol subtable. */
-    if ((subtable = this->find_subtable (3, 0)))
-    {
-      if (symbol) *symbol = true;
-      return subtable;
-    }
-
     /* Meh. */
     return &Null (CmapSubtable);
   }