> On Aug. 21, 2014, 5:34 p.m., Jie Yu wrote: > > 3rdparty/libprocess/src/process.cpp, line 1633 > > <https://reviews.apache.org/r/24939/diff/1/?file=666217#file666217line1633> > > > > Remove the space after inet_ntoa. > > > > inet_ntoa(addr.sin_addr) > > Vinod Kone wrote: > great. mind also adding this to "failed to initialize socket and > inet_pton" statements above?
Removed extra whitespace (probably style check should have alerted). Vinod, I didn't quite understand where should "this" be added so it would make sense :). - Nikita ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24939/#review51190 ----------------------------------------------------------- On Aug. 21, 2014, 6:22 p.m., Nikita Vetoshkin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24939/ > ----------------------------------------------------------- > > (Updated Aug. 21, 2014, 6:22 p.m.) > > > Review request for mesos. > > > Bugs: MESOS-1728 > https://issues.apache.org/jira/browse/MESOS-1728 > > > Repository: mesos-git > > > Description > ------- > > Report bind parameters on failure > > > Diffs > ----- > > 3rdparty/libprocess/src/process.cpp > ddcedb703cea67587c1c87993681686261107f47 > > Diff: https://reviews.apache.org/r/24939/diff/ > > > Testing > ------- > > Built master and slave and check new error message by issuing the same > commands in two separate consoles. > > > Thanks, > > Nikita Vetoshkin > >
