[HB] Fix warnings
authorBehdad Esfahbod <behdad@behdad.org>
Wed, 5 Aug 2009 01:42:23 +0000 (21:42 -0400)
committerBehdad Esfahbod <behdad@behdad.org>
Mon, 2 Nov 2009 19:40:30 +0000 (14:40 -0500)
src/hb-open-file-private.hh
src/hb-open-types-private.hh
src/hb-ot-layout-gsub-private.hh

index 5cb09da..e66b08a 100644 (file)
@@ -82,6 +82,7 @@ typedef struct OffsetTable
     for (unsigned int i = 0; i < count; i++)
       if (!SANITIZE_BASE (tableDir[i], base))
         return false;
+    return true;
   }
 
   private:
index 1af7c08..05619df 100644 (file)
@@ -484,6 +484,7 @@ struct GenericArrayOf
       if (!SANITIZE (array[i]))
         return false;
     */
+    return true;
   }
   inline bool sanitize (SANITIZE_ARG_DEF, const void *base) {
     if (!SANITIZE_GET_SIZE()) return false;
@@ -491,6 +492,7 @@ struct GenericArrayOf
     for (unsigned int i = 0; i < count; i++)
       if (!array[i].sanitize (SANITIZE_ARG, base))
         return false;
+    return true;
   }
   inline bool sanitize (SANITIZE_ARG_DEF, const void *base, const void *base2) {
     if (!SANITIZE_GET_SIZE()) return false;
@@ -498,6 +500,7 @@ struct GenericArrayOf
     for (unsigned int i = 0; i < count; i++)
       if (!array[i].sanitize (SANITIZE_ARG, base, base2))
         return false;
+    return true;
   }
   inline bool sanitize (SANITIZE_ARG_DEF, const void *base, unsigned int user_data) {
     if (!SANITIZE_GET_SIZE()) return false;
@@ -505,6 +508,7 @@ struct GenericArrayOf
     for (unsigned int i = 0; i < count; i++)
       if (!array[i].sanitize (SANITIZE_ARG, base, user_data))
         return false;
+    return true;
   }
 
   LenType len;
@@ -552,6 +556,7 @@ struct HeadlessArrayOf
       if (!SANITIZE (array[i]))
         return false;
     */
+    return true;
   }
 
   USHORT len;
index 9d35e99..4d21f65 100644 (file)
@@ -592,8 +592,7 @@ struct ReverseChainSingleSubstFormat1
     if (!SANITIZE_THIS (lookahead))
       return false;
     ArrayOf<GlyphID> &substitute = CAST (ArrayOf<GlyphID>, lookahead, lookahead.get_size ());
-    if (!SANITIZE (substitute))
-      return false;
+    return SANITIZE (substitute);
   }
 
   private: