[ovs-dev] [PATCH v2 12/14] db-ctl-base: Make 'cmd_show_tables' argument to ctl_init() optional.
Ben Pfaff
blp at nicira.com
Wed Sep 2 23:44:09 UTC 2015
This allows the client to omit the "show" command if it doesn't want it.
This will be used in an upcoming commit.
Signed-off-by: Ben Pfaff <blp at nicira.com>
---
lib/db-ctl-base.c | 17 ++++++++++++++---
1 file changed, 14 insertions(+), 3 deletions(-)
diff --git a/lib/db-ctl-base.c b/lib/db-ctl-base.c
index b018bad..cf49ac0 100644
--- a/lib/db-ctl-base.c
+++ b/lib/db-ctl-base.c
@@ -2036,10 +2036,15 @@ static const struct ctl_command_syntax db_ctl_commands[] = {
NULL, "--if-exists,--all", RW},
{"wait-until", 2, INT_MAX, "TABLE RECORD [COLUMN[:KEY]=VALUE]...",
pre_cmd_wait_until, cmd_wait_until, NULL, "", RO},
- {"show", 0, 0, "", pre_cmd_show, cmd_show, NULL, "", RO},
{NULL, 0, 0, NULL, NULL, NULL, NULL, NULL, RO},
};
+static void
+ctl_register_command(const struct ctl_command_syntax *command)
+{
+ shash_add_assert(&all_commands, command->name, command);
+}
+
/* Registers commands represented by 'struct ctl_command_syntax's to
* 'all_commands'. The last element of 'commands' must be an all-NULL
* element. */
@@ -2049,7 +2054,7 @@ ctl_register_commands(const struct ctl_command_syntax *commands)
const struct ctl_command_syntax *p;
for (p = commands; p->name; p++) {
- shash_add_assert(&all_commands, p->name, p);
+ ctl_register_command(p);
}
}
@@ -2060,9 +2065,15 @@ ctl_init(const struct ctl_table_class tables_[],
void (*ctl_exit_func_)(int status))
{
tables = tables_;
- cmd_show_tables = cmd_show_tables_;
ctl_exit_func = ctl_exit_func_;
ctl_register_commands(db_ctl_commands);
+
+ cmd_show_tables = cmd_show_tables_;
+ if (cmd_show_tables) {
+ static const struct ctl_command_syntax show =
+ {"show", 0, 0, "", pre_cmd_show, cmd_show, NULL, "", RO};
+ ctl_register_command(&show);
+ }
}
/* Returns the text for the database commands usage. */
--
2.1.3
More information about the dev
mailing list