[ovs-dev] [PATCH 3/4] ovn-nbctl: Give handler functions more specific names.

ALeX Wang ee07b291 at gmail.com
Sun Sep 13 16:54:03 UTC 2015


First three patches all look good to me,

Acked-by: Alex Wang <ee07b291 at gmail.com>

On 11 September 2015 at 20:29, Ben Pfaff <blp at nicira.com> wrote:

> I find that it's nice to give functions for commands names specific to the
> utility, even though they're static, because occasionally it makes it
> easier to find them using "tags", "grep", etc.
>
> Signed-off-by: Ben Pfaff <blp at nicira.com>
> ---
>  ovn/utilities/ovn-nbctl.c | 108
> +++++++++++++++++++++++-----------------------
>  1 file changed, 54 insertions(+), 54 deletions(-)
>
> diff --git a/ovn/utilities/ovn-nbctl.c b/ovn/utilities/ovn-nbctl.c
> index e684f73..6eae0e1 100644
> --- a/ovn/utilities/ovn-nbctl.c
> +++ b/ovn/utilities/ovn-nbctl.c
> @@ -177,7 +177,7 @@ print_lswitch(const struct nbrec_logical_switch
> *lswitch)
>  }
>
>  static void
> -do_show(struct ovs_cmdl_context *ctx)
> +nbctl_show(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const struct nbrec_logical_switch *lswitch;
> @@ -195,7 +195,7 @@ do_show(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lswitch_add(struct ovs_cmdl_context *ctx)
> +nbctl_lswitch_add(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      struct nbrec_logical_switch *lswitch;
> @@ -207,7 +207,7 @@ do_lswitch_add(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lswitch_del(struct ovs_cmdl_context *ctx)
> +nbctl_lswitch_del(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const char *id = ctx->argv[1];
> @@ -222,7 +222,7 @@ do_lswitch_del(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lswitch_list(struct ovs_cmdl_context *ctx)
> +nbctl_lswitch_list(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const struct nbrec_logical_switch *lswitch;
> @@ -243,7 +243,7 @@ do_lswitch_list(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lswitch_set_external_id(struct ovs_cmdl_context *ctx)
> +nbctl_lswitch_set_external_id(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const char *id = ctx->argv[1];
> @@ -267,7 +267,7 @@ do_lswitch_set_external_id(struct ovs_cmdl_context
> *ctx)
>  }
>
>  static void
> -do_lswitch_get_external_id(struct ovs_cmdl_context *ctx)
> +nbctl_lswitch_get_external_id(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const char *id = ctx->argv[1];
> @@ -328,7 +328,7 @@ lport_by_name_or_uuid(struct nbctl_context *nb_ctx,
> const char *id)
>  }
>
>  static void
> -do_lport_add(struct ovs_cmdl_context *ctx)
> +nbctl_lport_add(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      struct nbrec_logical_port *lport;
> @@ -395,7 +395,7 @@ remove_lport(const struct nbrec_logical_switch
> *lswitch, size_t idx)
>  }
>
>  static void
> -do_lport_del(struct ovs_cmdl_context *ctx)
> +nbctl_lport_del(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const struct nbrec_logical_port *lport;
> @@ -421,7 +421,7 @@ do_lport_del(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lport_list(struct ovs_cmdl_context *ctx)
> +nbctl_lport_list(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const char *id = ctx->argv[1];
> @@ -450,7 +450,7 @@ do_lport_list(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lport_get_parent(struct ovs_cmdl_context *ctx)
> +nbctl_lport_get_parent(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const struct nbrec_logical_port *lport;
> @@ -466,7 +466,7 @@ do_lport_get_parent(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lport_get_tag(struct ovs_cmdl_context *ctx)
> +nbctl_lport_get_tag(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const struct nbrec_logical_port *lport;
> @@ -482,7 +482,7 @@ do_lport_get_tag(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lport_set_external_id(struct ovs_cmdl_context *ctx)
> +nbctl_lport_set_external_id(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const char *id = ctx->argv[1];
> @@ -506,7 +506,7 @@ do_lport_set_external_id(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lport_get_external_id(struct ovs_cmdl_context *ctx)
> +nbctl_lport_get_external_id(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const char *id = ctx->argv[1];
> @@ -539,7 +539,7 @@ do_lport_get_external_id(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lport_set_macs(struct ovs_cmdl_context *ctx)
> +nbctl_lport_set_macs(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const char *id = ctx->argv[1];
> @@ -555,7 +555,7 @@ do_lport_set_macs(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lport_get_macs(struct ovs_cmdl_context *ctx)
> +nbctl_lport_get_macs(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const char *id = ctx->argv[1];
> @@ -581,7 +581,7 @@ do_lport_get_macs(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lport_set_port_security(struct ovs_cmdl_context *ctx)
> +nbctl_lport_set_port_security(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const char *id = ctx->argv[1];
> @@ -597,7 +597,7 @@ do_lport_set_port_security(struct ovs_cmdl_context
> *ctx)
>  }
>
>  static void
> -do_lport_get_port_security(struct ovs_cmdl_context *ctx)
> +nbctl_lport_get_port_security(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const char *id = ctx->argv[1];
> @@ -623,7 +623,7 @@ do_lport_get_port_security(struct ovs_cmdl_context
> *ctx)
>  }
>
>  static void
> -do_lport_get_up(struct ovs_cmdl_context *ctx)
> +nbctl_lport_get_up(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const char *id = ctx->argv[1];
> @@ -638,7 +638,7 @@ do_lport_get_up(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lport_set_enabled(struct ovs_cmdl_context *ctx)
> +nbctl_lport_set_enabled(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const char *id = ctx->argv[1];
> @@ -662,7 +662,7 @@ do_lport_set_enabled(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lport_get_enabled(struct ovs_cmdl_context *ctx)
> +nbctl_lport_get_enabled(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const char *id = ctx->argv[1];
> @@ -678,7 +678,7 @@ do_lport_get_enabled(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lport_set_type(struct ovs_cmdl_context *ctx)
> +nbctl_lport_set_type(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const char *id = ctx->argv[1];
> @@ -694,7 +694,7 @@ do_lport_set_type(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lport_get_type(struct ovs_cmdl_context *ctx)
> +nbctl_lport_get_type(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const char *id = ctx->argv[1];
> @@ -709,7 +709,7 @@ do_lport_get_type(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lport_set_options(struct ovs_cmdl_context *ctx)
> +nbctl_lport_set_options(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const char *id = ctx->argv[1];
> @@ -738,7 +738,7 @@ do_lport_set_options(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_lport_get_options(struct ovs_cmdl_context *ctx)
> +nbctl_lport_get_options(struct ovs_cmdl_context *ctx)
>  {
>      struct nbctl_context *nb_ctx = ctx->pvt;
>      const char *id = ctx->argv[1];
> @@ -793,7 +793,7 @@ acl_cmp(const void *acl1_, const void *acl2_)
>  }
>
>  static void
> -do_acl_list(struct ovs_cmdl_context *ctx)
> +nbctl_acl_list(struct ovs_cmdl_context *ctx)
>  {
>      const struct nbrec_logical_switch *lswitch;
>      struct nbctl_context *nb_ctx = ctx->pvt;
> @@ -822,7 +822,7 @@ do_acl_list(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_acl_add(struct ovs_cmdl_context *ctx)
> +nbctl_acl_add(struct ovs_cmdl_context *ctx)
>  {
>      const struct nbrec_logical_switch *lswitch;
>      struct nbctl_context *nb_ctx = ctx->pvt;
> @@ -880,7 +880,7 @@ do_acl_add(struct ovs_cmdl_context *ctx)
>  }
>
>  static void
> -do_acl_del(struct ovs_cmdl_context *ctx)
> +nbctl_acl_del(struct ovs_cmdl_context *ctx)
>  {
>      const struct nbrec_logical_switch *lswitch;
>      struct nbctl_context *nb_ctx = ctx->pvt;
> @@ -1022,112 +1022,112 @@ static const struct ovs_cmdl_command
> all_commands[] = {
>          .usage = "[LSWITCH]",
>          .min_args = 0,
>          .max_args = 1,
> -        .handler = do_show,
> +        .handler = nbctl_show,
>      },
>      {
>          .name = "lswitch-add",
>          .usage = "[LSWITCH]",
>          .min_args = 0,
>          .max_args = 1,
> -        .handler = do_lswitch_add,
> +        .handler = nbctl_lswitch_add,
>      },
>      {
>          .name = "lswitch-del",
>          .usage = "LSWITCH",
>          .min_args = 1,
>          .max_args = 1,
> -        .handler = do_lswitch_del,
> +        .handler = nbctl_lswitch_del,
>      },
>      {
>          .name = "lswitch-list",
>          .usage = "",
>          .min_args = 0,
>          .max_args = 0,
> -        .handler = do_lswitch_list,
> +        .handler = nbctl_lswitch_list,
>      },
>      {
>          .name = "lswitch-set-external-id",
>          .usage = "LSWITCH KEY [VALUE]",
>          .min_args = 2,
>          .max_args = 3,
> -        .handler = do_lswitch_set_external_id,
> +        .handler = nbctl_lswitch_set_external_id,
>      },
>      {
>          .name = "lswitch-get-external-id",
>          .usage = "LSWITCH [KEY]",
>          .min_args = 1,
>          .max_args = 2,
> -        .handler = do_lswitch_get_external_id,
> +        .handler = nbctl_lswitch_get_external_id,
>      },
>      {
>          .name = "acl-add",
>          .usage = "LSWITCH DIRECTION PRIORITY MATCH ACTION [log]",
>          .min_args = 5,
>          .max_args = 6,
> -        .handler = do_acl_add,
> +        .handler = nbctl_acl_add,
>      },
>      {
>          .name = "acl-del",
>          .usage = "LSWITCH [DIRECTION [PRIORITY MATCH]]",
>          .min_args = 1,
>          .max_args = 4,
> -        .handler = do_acl_del,
> +        .handler = nbctl_acl_del,
>      },
>      {
>          .name = "acl-list",
>          .usage = "LSWITCH",
>          .min_args = 1,
>          .max_args = 1,
> -        .handler = do_acl_list,
> +        .handler = nbctl_acl_list,
>      },
>      {
>          .name = "lport-add",
>          .usage = "LSWITCH LPORT [PARENT] [TAG]",
>          .min_args = 2,
>          .max_args = 4,
> -        .handler = do_lport_add,
> +        .handler = nbctl_lport_add,
>      },
>      {
>          .name = "lport-del",
>          .usage = "LPORT",
>          .min_args = 1,
>          .max_args = 1,
> -        .handler = do_lport_del,
> +        .handler = nbctl_lport_del,
>      },
>      {
>          .name = "lport-list",
>          .usage = "LSWITCH",
>          .min_args = 1,
>          .max_args = 1,
> -        .handler = do_lport_list,
> +        .handler = nbctl_lport_list,
>      },
>      {
>          .name = "lport-get-parent",
>          .usage = "LPORT",
>          .min_args = 1,
>          .max_args = 1,
> -        .handler = do_lport_get_parent,
> +        .handler = nbctl_lport_get_parent,
>      },
>      {
>          .name = "lport-get-tag",
>          .usage = "LPORT",
>          .min_args = 1,
>          .max_args = 1,
> -        .handler = do_lport_get_tag,
> +        .handler = nbctl_lport_get_tag,
>      },
>      {
>          .name = "lport-set-external-id",
>          .usage = "LPORT KEY [VALUE]",
>          .min_args = 2,
>          .max_args = 3,
> -        .handler = do_lport_set_external_id,
> +        .handler = nbctl_lport_set_external_id,
>      },
>      {
>          .name = "lport-get-external-id",
>          .usage = "LPORT [KEY]",
>          .min_args = 1,
>          .max_args = 2,
> -        .handler = do_lport_get_external_id,
> +        .handler = nbctl_lport_get_external_id,
>      },
>      {
>          .name = "lport-set-macs",
> @@ -1135,14 +1135,14 @@ static const struct ovs_cmdl_command
> all_commands[] = {
>          .min_args = 1,
>          /* Accept however many arguments the system will allow. */
>          .max_args = INT_MAX,
> -        .handler = do_lport_set_macs,
> +        .handler = nbctl_lport_set_macs,
>      },
>      {
>          .name = "lport-get-macs",
>          .usage = "LPORT",
>          .min_args = 1,
>          .max_args = 1,
> -        .handler = do_lport_get_macs,
> +        .handler = nbctl_lport_get_macs,
>      },
>      {
>          .name = "lport-set-port-security",
> @@ -1150,63 +1150,63 @@ static const struct ovs_cmdl_command
> all_commands[] = {
>          .min_args = 0,
>          /* Accept however many arguments the system will allow. */
>          .max_args = INT_MAX,
> -        .handler = do_lport_set_port_security,
> +        .handler = nbctl_lport_set_port_security,
>      },
>      {
>          .name = "lport-get-port-security",
>          .usage = "LPORT",
>          .min_args = 1,
>          .max_args = 1,
> -        .handler = do_lport_get_port_security,
> +        .handler = nbctl_lport_get_port_security,
>      },
>      {
>          .name = "lport-get-up",
>          .usage = "LPORT",
>          .min_args = 1,
>          .max_args = 1,
> -        .handler = do_lport_get_up,
> +        .handler = nbctl_lport_get_up,
>      },
>      {
>          .name = "lport-set-enabled",
>          .usage = "LPORT STATE",
>          .min_args = 2,
>          .max_args = 2,
> -        .handler = do_lport_set_enabled,
> +        .handler = nbctl_lport_set_enabled,
>      },
>      {
>          .name = "lport-get-enabled",
>          .usage = "LPORT",
>          .min_args = 1,
>          .max_args = 1,
> -        .handler = do_lport_get_enabled,
> +        .handler = nbctl_lport_get_enabled,
>      },
>      {
>          .name = "lport-set-type",
>          .usage = "LPORT TYPE",
>          .min_args = 2,
>          .max_args = 2,
> -        .handler = do_lport_set_type,
> +        .handler = nbctl_lport_set_type,
>      },
>      {
>          .name = "lport-get-type",
>          .usage = "LPORT",
>          .min_args = 1,
>          .max_args = 1,
> -        .handler = do_lport_get_type,
> +        .handler = nbctl_lport_get_type,
>      },
>      {
>          .name = "lport-set-options",
>          .usage = "LPORT KEY=VALUE [KEY=VALUE]...",
>          .min_args = 1,
>          .max_args = INT_MAX,
> -        .handler = do_lport_set_options
> +        .handler = nbctl_lport_set_options
>      },
>      {
>          .name = "lport-get-options",
>          .usage = "LPORT",
>          .min_args = 1,
>          .max_args = 1,
> -        .handler = do_lport_get_options,
> +        .handler = nbctl_lport_get_options,
>      },
>
>      {
> --
> 2.1.3
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>



-- 
Alex Wang,
Open vSwitch developer



More information about the dev mailing list