diff src/filefilter.c @ 1315:c1d108ab3388

Since READ_*() macros are now exported, do not hide the if() continue; statement in them. Wrap WRITE_STRING() instructions in a safer do {} while (0).

--- a/src/filefilter.c	Mon Feb 23 22:13:09 2009 +0000
+++ b/src/filefilter.c	Tue Feb 24 17:47:45 2009 +0000
@@ -460,13 +460,13 @@
 		const gchar *option = *attribute_names++;
 		const gchar *value = *attribute_values++;
 
-		READ_CHAR(fe, key);
-		READ_BOOL(fe, enabled);
-		READ_CHAR(fe, extensions);
-		READ_CHAR(fe, description);
-		READ_UINT(fe, file_class);
-		READ_BOOL(fe, writable);
-		READ_BOOL(fe, allow_sidecar);
+		if (READ_CHAR(fe, key)) continue;
+		if (READ_BOOL(fe, enabled)) continue;
+		if (READ_CHAR(fe, extensions)) continue;
+		if (READ_CHAR(fe, description)) continue;
+		if (READ_UINT(fe, file_class)) continue;
+		if (READ_BOOL(fe, writable)) continue;
+		if (READ_BOOL(fe, allow_sidecar)) continue;
 
 		printf("unknown attribute %s = %s\n", option, value);
 		}