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

Ben Pfaff blp at nicira.com
Thu Sep 29 21:36:14 UTC 2011


Thanks, I pushed this.

On Thu, Sep 29, 2011 at 02:17:22PM -0700, Ethan Jackson wrote:
> 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