Minor

diff --git a/src/hb-ot-layout-common-private.hh b/src/hb-ot-layout-common-private.hh
index fb7fd3b..db42894 100644 (file)
--- a/src/hb-ot-layout-common-private.hh
+++ b/src/hb-ot-layout-common-private.hh
@@ -34,7 +34,7 @@
 #include "hb-set-private.hh"
 
 
-#define NOT_COVERED            ((unsigned int) 0x110000)
+#define NOT_COVERED            ((unsigned int) -1)
 #define MAX_NESTING_LEVEL      8
 
 
@@ -348,9 +348,8 @@ struct CoverageFormat1
   inline unsigned int get_coverage (hb_codepoint_t glyph_id) const
   {
     int i = glyphArray.search (glyph_id);
-    if (i != -1)
-        return i;
-    return NOT_COVERED;
+    ASSERT_STATIC (((unsigned int) -1) == NOT_COVERED);
+    return i;
   }
 
   inline bool sanitize (hb_sanitize_context_t *c) {

diff --git a/src/hb-ot-layout-gpos-table.hh b/src/hb-ot-layout-gpos-table.hh
index 6d05c99..3b1ae2a 100644 (file)
--- a/src/hb-ot-layout-gpos-table.hh
+++ b/src/hb-ot-layout-gpos-table.hh
@@ -1533,7 +1533,7 @@ struct PosLookup : Lookup
       while (c->buffer->idx < c->buffer->len)
       {
        if ((c->buffer->cur().mask & c->lookup_mask) &&
-           (*coverage) (c->buffer->cur().codepoint) != NOT_COVERED &&
+           coverage->get_coverage (c->buffer->cur().codepoint) != NOT_COVERED &&
            apply_once (c))
          ret = true;
        else

diff --git a/src/hb-ot-layout-gsub-table.hh b/src/hb-ot-layout-gsub-table.hh
index e50ecd4..2cbab32 100644 (file)
--- a/src/hb-ot-layout-gsub-table.hh
+++ b/src/hb-ot-layout-gsub-table.hh
@@ -1210,7 +1210,7 @@ struct SubstLookup : Lookup
          while (c->buffer->idx < c->buffer->len)
          {
            if ((c->buffer->cur().mask & c->lookup_mask) &&
-               (*coverage) (c->buffer->cur().codepoint) != NOT_COVERED &&
+               coverage->get_coverage (c->buffer->cur().codepoint) != NOT_COVERED &&
                apply_once (c))
              ret = true;
            else