[ovs-dev] [PATCH 3/4] db-ctl-base: remove ctl_get_all_commands() function
Andy Zhou
azhou at nicira.com
Tue Jul 14 06:48:03 UTC 2015
all_commands currently is some times accessed directly, some times via
a function call. Change to always access it directly.
Signed-off-by: Andy Zhou <azhou at nicira.com>
---
lib/db-ctl-base.c | 14 ++++----------
lib/db-ctl-base.h | 1 -
2 files changed, 4 insertions(+), 11 deletions(-)
diff --git a/lib/db-ctl-base.c b/lib/db-ctl-base.c
index b0f6fd5..e3ba0c5 100644
--- a/lib/db-ctl-base.c
+++ b/lib/db-ctl-base.c
@@ -1776,7 +1776,7 @@ ctl_add_cmd_options(struct option **options_p, size_t *n_options_p,
const struct shash_node *node;
size_t n_existing_options = *n_options_p;
- SHASH_FOR_EACH (node, ctl_get_all_commands()) {
+ SHASH_FOR_EACH (node, &all_commands) {
const struct ctl_command_syntax *p = node->data;
if (p->options[0]) {
@@ -1866,7 +1866,7 @@ ctl_print_commands(void)
{
const struct shash_node *node;
- SHASH_FOR_EACH (node, ctl_get_all_commands()) {
+ SHASH_FOR_EACH (node, &all_commands) {
const struct ctl_command_syntax *p = node->data;
char *options = xstrdup(p->options);
char *options_begin = options;
@@ -1922,7 +1922,8 @@ bool
ctl_might_write_to_db(char **argv)
{
for (; *argv; argv++) {
- const struct ctl_command_syntax *p = shash_find_data(&all_commands, *argv);
+ const struct ctl_command_syntax *p = shash_find_data(&all_commands,
+ *argv);
if (p && p->mode == RW) {
return true;
}
@@ -2013,13 +2014,6 @@ ctl_init(const struct ctl_table_class tables_[],
ctl_register_commands(db_ctl_commands);
}
-/* Returns 'all_commands'. */
-const struct shash *
-ctl_get_all_commands(void)
-{
- return &all_commands;
-}
-
/* Returns the text for the database commands usage. */
const char *
ctl_get_db_cmd_usage(void)
diff --git a/lib/db-ctl-base.h b/lib/db-ctl-base.h
index a361b50..e750599 100644
--- a/lib/db-ctl-base.h
+++ b/lib/db-ctl-base.h
@@ -151,7 +151,6 @@ void ctl_print_options(const struct option *);
void ctl_add_cmd_options(struct option **, size_t *n_options_p,
size_t *allocated_options_p, int opt_val);
void ctl_register_commands(const struct ctl_command_syntax *);
-const struct shash *ctl_get_all_commands(void);
struct ctl_command *ctl_parse_commands(int argc, char *argv[],
struct shash *local_options,
size_t *n_commandsp);
--
1.9.1
More information about the dev
mailing list