ovsdb-parser: Use sset instead of svec for detecting unused members.
authorBen Pfaff <blp@nicira.com>
Fri, 25 Mar 2011 20:20:35 +0000 (13:20 -0700)
committerBen Pfaff <blp@nicira.com>
Thu, 31 Mar 2011 23:42:01 +0000 (16:42 -0700)
Should be slightly cheaper than sorting a list (O(n) vs. O(n lg n)).

lib/ovsdb-parser.c
lib/ovsdb-parser.h

index 2a4c3d99e89abd06a8c10f7cfe382712b2d96e39..e1832a999fce0d77cecbcd6e9cf6234bbbd0b2f9 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (c) 2009 Nicira Networks
+/* Copyright (c) 2009, 2011 Nicira Networks
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -32,7 +32,7 @@ ovsdb_parser_init(struct ovsdb_parser *parser, const struct json *json,
     parser->name = xvasprintf(name, args);
     va_end(args);
 
-    svec_init(&parser->used);
+    sset_init(&parser->used);
     parser->error = NULL;
 
     parser->json = (json && json->type == JSON_OBJECT ? json : NULL);
@@ -85,7 +85,7 @@ ovsdb_parser_member(struct ovsdb_parser *parser, const char *name,
         || (types & OP_ID && value->type == JSON_STRING
             && ovsdb_parser_is_id(value->u.string)))
     {
-        svec_add(&parser->used, name);
+        sset_add(&parser->used, name);
         return value;
     } else {
         ovsdb_parser_raise_error(parser, "Type mismatch for member '%s'.",
@@ -133,14 +133,12 @@ ovsdb_parser_finish(struct ovsdb_parser *parser)
         const struct shash *object = json_object(parser->json);
         size_t n_unused;
 
-        /* XXX this method of detecting unused members can be made cheaper */
-        svec_sort_unique(&parser->used);
-        n_unused = shash_count(object) - parser->used.n;
+        n_unused = shash_count(object) - sset_count(&parser->used);
         if (n_unused) {
             struct shash_node *node;
 
             SHASH_FOR_EACH (node, object) {
-                if (!svec_contains(&parser->used, node->name)) {
+                if (!sset_contains(&parser->used, node->name)) {
                     if (n_unused > 1) {
                         ovsdb_parser_raise_error(
                             parser,
@@ -160,7 +158,7 @@ ovsdb_parser_finish(struct ovsdb_parser *parser)
     }
 
     free(parser->name);
-    svec_destroy(&parser->used);
+    sset_destroy(&parser->used);
 
     return parser->error;
 }
index d6270bb12cb72cbec0452b7d6644f2d204ac0be4..44586aa7f3b69a05fc146a1617dabd3bc34e4fa2 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (c) 2009, 2010 Nicira Networks
+/* Copyright (c) 2009, 2010, 2011 Nicira Networks
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
 #include <stdbool.h>
 #include "compiler.h"
 #include "json.h"
-#include "svec.h"
+#include "sset.h"
 #include "util.h"
 
 struct ovsdb_parser {
     char *name;                 /* Used only in error messages. */
-    struct svec used;           /* Already-parsed names from 'object'. */
+    struct sset used;           /* Already-parsed names from 'object'. */
     const struct json *json;    /* JSON object being parsed. */
     struct ovsdb_error *error;  /* Error signaled, if any. */
 };