ovsdb-parser: Fix logic in ovsdb_parser_member().

I don't know why this test was written as it was.  The new test makes more
sense, and may be more correct as well.

diff --git a/lib/ovsdb-parser.c b/lib/ovsdb-parser.c
index d923d21a88897f48f20c206f4f0d758f424a814f..5419467c71068a719b262fee70789a415d0c03fb 100644 (file)
--- a/lib/ovsdb-parser.c
+++ b/lib/ovsdb-parser.c
@@ -80,11 +80,10 @@ ovsdb_parser_member(struct ovsdb_parser *parser, const char *name,
         return NULL;
     }
 
-    if (value->type >= 0 && value->type < JSON_N_TYPES
-        && (types & (1u << value->type)
-            || (types & OP_ID
-                && value->type == JSON_STRING
-                && is_id(value->u.string))))
+    if ((value->type >= 0 && value->type < JSON_N_TYPES
+         && types & (1u << value->type))
+        || (types & OP_ID && value->type == JSON_STRING
+            && is_id(value->u.string)))
     {
         svec_add(&parser->used, name);
         return value;