Thanks for the review.
On Tue, Apr 02, 2013 at 09:27:12PM -0700, Andy Zhou wrote:
> Looks good.
>
> On Tuesday, April 2, 2013, Ben Pfaff wrote:
>
> > I spotted this in a case where I ran ovsdb-server, by hand, on an empty
> > database.
> >
> > Signed-off-by: Ben Pfaff <b...@nicira.com <javascript:;>>
> > ---
> > lib/memory.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/lib/memory.c b/lib/memory.c
> > index 1babfe9..274f307 100644
> > --- a/lib/memory.c
> > +++ b/lib/memory.c
> > @@ -1,5 +1,5 @@
> > /*
> > - * Copyright (c) 2012 Nicira, Inc.
> > + * Copyright (c) 2012, 2013 Nicira, Inc.
> > *
> > * Licensed under the Apache License, Version 2.0 (the "License");
> > * you may not use this file except in compliance with the License.
> > @@ -137,7 +137,7 @@ memory_report(const struct simap *usage)
> > ds_init(&s);
> > compose_report(usage, &s);
> >
> > - if (want_report) {
> > + if (want_report && s.length > 0) {
> > VLOG_INFO("%s", ds_cstr(&s));
> > want_report = false;
> > }
> > --
> > 1.7.10.4
> >
> > _______________________________________________
> > dev mailing list
> > dev@openvswitch.org <javascript:;>
> > http://openvswitch.org/mailman/listinfo/dev
> >
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
