diff options
| author | Ben Pfaff <blp@ovn.org> | 2018-01-31 11:23:24 -0800 |
|---|---|---|
| committer | Ben Pfaff <blp@ovn.org> | 2018-02-01 11:21:34 -0800 |
| commit | 500db308e2704fa9fc5ff69cdcd9cb3b22116e99 (patch) | |
| tree | 8a5cf13bf33232372b806bcbd48c791add83a4df /ovsdb/ovsdb-server.c | |
| parent | 4bc938ccb3124282c873084a7d11cd8bc37e27a7 (diff) | |
| download | openvswitch-500db308e2704fa9fc5ff69cdcd9cb3b22116e99.tar.gz | |
util: Document and rely on ovs_assert() always evaluating its argument.
The ovs_assert() macro always evaluates its argument, even when NDEBUG is
defined so that failure is ignored. This behavior wasn't documented, and
thus a lot of code didn't rely on it. This commit documents the behavior
and simplifies bits of code that heretofore didn't rely on it.
Signed-off-by: Ben Pfaff <blp@ovn.org>
Reviewed-by: Yifeng Sun <pkusunyifeng@gmail.com>
Diffstat (limited to 'ovsdb/ovsdb-server.c')
| -rw-r--r-- | ovsdb/ovsdb-server.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/ovsdb/ovsdb-server.c b/ovsdb/ovsdb-server.c index 7f2d19ef5..3ac7bf472 100644 --- a/ovsdb/ovsdb-server.c +++ b/ovsdb/ovsdb-server.c @@ -1291,11 +1291,9 @@ static void remove_db(struct server_config *config, struct shash_node *node) { struct db *db; - bool ok; db = node->data; - ok = ovsdb_jsonrpc_server_remove_db(config->jsonrpc, db->db); - ovs_assert(ok); + ovs_assert(ovsdb_jsonrpc_server_remove_db(config->jsonrpc, db->db)); close_db(db); shash_delete(config->all_dbs, node); |