[ovs-dev] [PATCH] vlog: Accept any case for ANY and PATTERN when setting levels.

Ethan Jackson ethan at nicira.com
Thu Sep 29 21:17:22 UTC 2011


Looks good.

Ethan

On Tue, Sep 27, 2011 at 12:23, Ben Pfaff <blp at nicira.com> wrote:
> ---
>  lib/vlog.c |    6 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/lib/vlog.c b/lib/vlog.c
> index ef95b1b..1fe2383 100644
> --- a/lib/vlog.c
> +++ b/lib/vlog.c
> @@ -346,7 +346,7 @@ vlog_set_levels_from_string(const char *s_)
>
>         facility = strtok_r(NULL, ":", &save_ptr);
>
> -        if (!facility || !strcmp(facility, "ANY")) {
> +        if (!facility || !strcasecmp(facility, "ANY")) {
>             e_facility = VLF_ANY_FACILITY;
>         } else {
>             e_facility = vlog_get_facility_val(facility);
> @@ -357,14 +357,14 @@ vlog_set_levels_from_string(const char *s_)
>             }
>         }
>
> -        if (!strcmp(module, "PATTERN")) {
> +        if (!strcasecmp(module, "PATTERN")) {
>             vlog_set_pattern(e_facility, save_ptr);
>             break;
>         } else {
>             char *level;
>             enum vlog_level e_level;
>
> -            if (!strcmp(module, "ANY")) {
> +            if (!strcasecmp(module, "ANY")) {
>                 e_module = NULL;
>             } else {
>                 e_module = vlog_module_from_name(module);
> --
> 1.7.4.4
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>



More information about the dev mailing list