> On Jan. 6, 2015, 5:37 p.m., Dominic Hamon wrote: > > 3rdparty/libprocess/include/process/socket.hpp, line 49 > > <https://reviews.apache.org/r/29289/diff/4/?file=807771#file807771line49> > > > > the API is more complex here, which seems like a regression. Is it > > possible to simplify this? > > > > sockaddr_in addr = net::createSockaddrIn(net::IP::Address(), 0); > > > > ? > > > > though I suppose actually the accept call should be wrapped up now and > > use the Address abstraction, so maybe this is a temporary change?
the part with initializing sockaddr_in can be omitted. - Evelina ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29289/#review66865 ----------------------------------------------------------- On Jan. 6, 2015, 2:17 p.m., Evelina Dumitrescu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29289/ > ----------------------------------------------------------- > > (Updated Jan. 6, 2015, 2:17 p.m.) > > > Review request for mesos, Benjamin Hindman, Dominic Hamon, Joris Van > Remoortere, and Niklas Nielsen. > > > Bugs: MESOS-1919 > https://issues.apache.org/jira/browse/MESOS-1919 > > > Repository: mesos-git > > > Description > ------- > > Created the inner class Address encapsulated inside the IP class. > The class uses a union with the in_addr and in6_addr fields. > I considered that the The MasterInfo protobuffers should have both an ipv4 > and an ipv6 field. > I intend to use the same Classifiers, addition, removal and update of > container filters, but write different encode/decode functions for IPv4/ICMP > and IPv6/ICMPv6 because the processing of the protocol headers differ. > > > Diffs > ----- > > 3rdparty/libprocess/include/process/node.hpp > 173eb8abe0d0930c92f261be7a95e07b1d4c16af > 3rdparty/libprocess/include/process/pid.hpp > 7dccf297ea473a9b19dd450f02c7ae4fd0c60a22 > 3rdparty/libprocess/include/process/socket.hpp > ddb9e365fc1e65a568bdac4973964df1ab8cc05e > 3rdparty/libprocess/src/pid.cpp 085e0b9abe9a9f33a63247915835decbf942274d > 3rdparty/libprocess/src/process.cpp > 67b6b3b9c13d95fa1a24b48a12c5c831c7f249bf > > Diff: https://reviews.apache.org/r/29289/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Evelina Dumitrescu > >
