From: Behdad Esfahbod <behdad@behdad.org>
Date: Sat, 28 Jul 2012 22:03:20 +0000 (-0400)
Subject: Minor
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=dadede012e4841f9fcb70d514fdc752f3ea4663d;p=platform%2Fupstream%2FlibHarfBuzzSharp.git

Minor
---

diff --git a/src/hb-ot-layout-common-private.hh b/src/hb-ot-layout-common-private.hh
index fb7fd3b..db42894 100644
--- 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
--- 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
--- 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