Re: [Linuxptp-devel] [PATCH 4/5] Common type holding an address

2014-04-22 Thread Jiri Benc
On Thu, 17 Apr 2014 07:19:05 +0200, Richard Cochran wrote: > I don't see why you need to call into the transport twice. > > Can't you pass addr=NULL in order to get the default address? Makes sense. Jiri -- Jiri Benc ---

Re: [Linuxptp-devel] [PATCH 4/5] Common type holding an address

2014-04-16 Thread Richard Cochran
Jiri, I think this series brings a nice improvement, on second look. But the change in this patch could be better yet? See below... On Fri, Apr 11, 2014 at 12:25:54PM +0200, Jiri Benc wrote: > diff --git a/transport.c b/transport.c > index cb799a68aa9f..25d569607f71 100644 > --- a/transport.c >

Re: [Linuxptp-devel] [PATCH 4/5] Common type holding an address

2014-04-11 Thread Richard Cochran
On Fri, Apr 11, 2014 at 06:13:22PM +0200, Jiri Benc wrote: > > It's the first member of the union. I originally had only struct > sockaddr_storage here but it led to horrible typecasting in the code > (which is quite error prone when you accidentally put an extra & before > the variable). In the e

Re: [Linuxptp-devel] [PATCH 4/5] Common type holding an address

2014-04-11 Thread Jiri Benc
On Fri, 11 Apr 2014 14:08:16 +0200, Richard Cochran wrote: > > +struct address { > > + socklen_t len; > > + union { > > + struct sockaddr_storage ss; > > + struct sockaddr_in sin; > > + struct sockaddr_in6 sin6; > > + struct sockaddr_un sun; > > +

Re: [Linuxptp-devel] [PATCH 4/5] Common type holding an address

2014-04-11 Thread Richard Cochran
On Fri, Apr 11, 2014 at 12:25:54PM +0200, Jiri Benc wrote: > This modifies all transports to use a new common address type, struct > address. This address is stored in a ptp_message for all received messages. > > For sending, the "default" address is used with the default sending > functions, tran

[Linuxptp-devel] [PATCH 4/5] Common type holding an address

2014-04-11 Thread Jiri Benc
This modifies all transports to use a new common address type, struct address. This address is stored in a ptp_message for all received messages. For sending, the "default" address is used with the default sending functions, transport_send and transport_peer. The default address depends on the tra