Fix missing negation in unreachable code!
authorBehdad Esfahbod <behdad@behdad.org>
Tue, 12 Oct 2010 15:17:30 +0000 (11:17 -0400)
committerBehdad Esfahbod <behdad@behdad.org>
Tue, 12 Oct 2010 15:17:30 +0000 (11:17 -0400)
Mozilla bug #603346

src/hb-open-type-private.hh

index ea1d371..38ba3a3 100644 (file)
@@ -587,7 +587,7 @@ struct GenericArrayOf
      * other structs. */
     unsigned int count = len;
     for (unsigned int i = 0; i < count; i++)
-      if (array[i].sanitize (c))
+      if (unlikely (!array[i].sanitize (c)))
         return false;
     return true;
   }
@@ -697,9 +697,8 @@ struct HeadlessArrayOf
      * to do have a simple sanitize(), ie. they do not reference
      * other structs. */
     unsigned int count = len ? len - 1 : 0;
-    Type *a = array;
     for (unsigned int i = 0; i < count; i++)
-      if (unlikely (!a[i].sanitize (c)))
+      if (unlikely (!array[i].sanitize (c)))
         return false;
     return true;
   }