On Wed, Dec 11, 2013 at 10:24:50AM -0800, Jarno Rajahalme wrote: > > On Dec 10, 2013, at 11:20 PM, Ben Pfaff <b...@nicira.com> wrote: > > > This allows its callers to avoid duplicating the code. > > > > Signed-off-by: Ben Pfaff <b...@nicira.com> > > --- > > lib/netdev.c | 10 ++++++++-- > > utilities/ovs-dpctl.c | 2 -- > > vswitchd/bridge.c | 9 +-------- > > 3 files changed, 9 insertions(+), 12 deletions(-) > > > > diff --git a/lib/netdev.c b/lib/netdev.c > > index 5ed6062..159ca51 100644 > > --- a/lib/netdev.c > > +++ b/lib/netdev.c > > @@ -386,8 +386,14 @@ netdev_set_config(struct netdev *netdev, const struct > > smap *args) > > { > > if (netdev->netdev_class->set_config) { > > const struct smap no_args = SMAP_INITIALIZER(&no_args); > > - return netdev->netdev_class->set_config(netdev, > > - args ? args : &no_args); > > + int error; > > + > > + error = netdev->netdev_class->set_config(netdev, > > + args ? args : &no_args); > > + if (error) { > > + VLOG_WARN("%s: could not set configuration (%s)", > > + netdev_get_name(netdev), ovs_strerror(error)); > > + } > > Wouldn?t you still want to return the error? The callers seem to expect it.
Yes, good point, sorry. I added a "return error;" here. > Otherwise: > > Acked-by: Jarno Rajahalme <jrajaha...@nicira.com> Thanks! _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev