Fix confusion between boolean and FT_Error return. (GSUB equivalent of fix
authorOwen Taylor <otaylor@redhat.com>
Sun, 27 Jul 2003 01:10:15 +0000 (01:10 +0000)
committerOwen Taylor <otaylor@src.gnome.org>
Sun, 27 Jul 2003 01:10:15 +0000 (01:10 +0000)
Sat Jul 26 21:06:26 2003  Owen Taylor  <otaylor@redhat.com>

        * pango/opentype/ftxgsub.c (Load_EmptyOrClassDefinition):
        Fix confusion between boolean and FT_Error return.
        (GSUB equivalent of fix for #108358)

src/ftxgsub.c

index aedc551..ff42dc6 100644 (file)
     if ( class_offset )
       {
         if ( !FILE_Seek( class_offset + base_offset ) )
-          error = Load_ClassDefinition( cd, limit, stream ) == TT_Err_Ok;
+          error = Load_ClassDefinition( cd, limit, stream );
       }
     else
        error = Load_EmptyClassDefinition ( cd, stream );
 
-    (void)FILE_Seek( cur_offset );
+    if (error == TT_Err_Ok)
+      (void)FILE_Seek( cur_offset ); /* Changes error as a side-effect */
 
     return error;
   }