Fix checks in gfield.c
authorOwen W. Taylor <otaylor@fishsoup.net>
Tue, 21 Apr 2009 14:57:50 +0000 (10:57 -0400)
committerOwen W. Taylor <otaylor@fishsoup.net>
Thu, 9 Jul 2009 10:06:20 +0000 (11:06 +0100)
When checking if the readable/writable flags are missing for the fields
we are trying to read and write, use (a & flag) == 0, not (!a & flag).

http://bugzilla.gnome.org/show_bug.cgi?id=579727

girepository/gfield.c

index 2fa982275cd220ec6553c2c9a48811b0bf2459a0..94c2fb5976f07c5bd154b674ef09ddc8c2f2df93 100644 (file)
@@ -25,7 +25,7 @@ g_field_info_get_field (GIFieldInfo *field_info,
     GITypeInfo *type_info;
     gboolean result = FALSE;
 
-    if (!g_field_info_get_flags (field_info) & GI_FIELD_IS_READABLE)
+    if ((g_field_info_get_flags (field_info) & GI_FIELD_IS_READABLE) == 0)
       return FALSE;
 
     offset = g_field_info_get_offset (field_info);
@@ -219,7 +219,7 @@ g_field_info_get_field (GIFieldInfo *field_info,
  * to write fields where memory management would by required. A field
  * with a type such as 'char *' must be set with a setter function.
  *
- * Returns: %TRUE if reading the field succeeded, otherwise %FALSE
+ * Returns: %TRUE if writing the field succeeded, otherwise %FALSE
  */
 gboolean
 g_field_info_set_field (GIFieldInfo     *field_info,
@@ -230,7 +230,7 @@ g_field_info_set_field (GIFieldInfo     *field_info,
     GITypeInfo *type_info;
     gboolean result = FALSE;
 
-    if (!g_field_info_get_flags (field_info) & GI_FIELD_IS_WRITABLE)
+    if ((g_field_info_get_flags (field_info) & GI_FIELD_IS_WRITABLE) == 0)
       return FALSE;
 
     offset = g_field_info_get_offset (field_info);