On Mon, Apr 11, 2016 at 6:12 PM, Ben Pfaff <[email protected]> wrote: > On Sun, Apr 03, 2016 at 09:49:03PM -0400, Ramu Ramamurthy wrote: >> Move the function extract_lport_addresses to a file >> in ovn/lib since that function can be used by ovn-controller also >> to parse addresses stored in the mac column of the >> port_binding table. Currently that function is used only >> in ovn_northd. >> >> Signed-off-by: Ramu Ramamurthy <[email protected]> > > Please add the standard license notice at the top of each of the new > files. > > Function-level comments go on function definitions, not their header > file prototypes. > > The comment > +/* contains ovn utility functions */ > isn't very useful and I'd recommend deleting it. > > Thanks, > > Ben.
Ben, Thanks for the review, I will update in the next version with changes noted above. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
