Pushed with changes suggested. Thanks a lot for the review.
On Fri, Dec 6, 2013 at 2:46 PM, Joe Stringer <joestrin...@nicira.com> wrote: > On 6 December 2013 14:27, Andy Zhou <az...@nicira.com> wrote: > > diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c > > index 0f4cf2c..d64de65 100644 > > --- a/vswitchd/bridge.c > > +++ b/vswitchd/bridge.c > > @@ -21,7 +21,7 @@ > > #include "async-append.h" > > #include "bfd.h" > > #include "bitmap.h" > > -#include "bond.h" > > +#include "ofproto/bond.h" > > #include "cfm.h" > > #include "coverage.h" > > #include "daemon.h" > > It seems like these includes are in alphabetical order. The > replacement include could be put next to the other ofproto/ include. > > There is also a reference to lib/bond.c in PORTING. > > Otherwise, looks good to me. >
_______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev