On Sat, 3 May 2014 20:49:16 +0200, Richard Cochran wrote: > On Fri, May 02, 2014 at 12:37:51PM +0200, Jiri Benc wrote: > > Split management message creation to more fine-grained functions to allow > > notification messages to be created. > > I must admit I didn't understand what you did, at first reading. Can > you lose a few words about how you split the code and why? > > Is it that the 'fill' method is shared by regular management replies > and by the event notifiers?
Yes. And I wanted to avoid code duplication, thus I extracted the generic part of port_management_get_response to a separate function. The new port_management_fill_response is called from port_management_get_response (so the function behaves exactly the same as before this patch) and from a new port_notify_event function. The difference is port_management_get_response uses the request message to construct the reply message, while port_notify_event constructs the reply message based on the notification id. > > +struct ptp_message *port_management_reply(struct PortIdentity pid, > > + struct port *ingress, > > + struct ptp_message *req) > > +{ > > + UInteger8 boundaryHops; > > + > > + boundaryHops = req->management.startingBoundaryHops - > > req->management.boundaryHops; > > Line too long. Will split it. Jiri -- Jiri Benc ------------------------------------------------------------------------------ Is your legacy SCM system holding you back? Join Perforce May 7 to find out: • 3 signs your SCM is hindering your productivity • Requirements for releasing software faster • Expert tips and advice for migrating your SCM now http://p.sf.net/sfu/perforce _______________________________________________ Linuxptp-devel mailing list Linuxptp-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linuxptp-devel