> On June 23, 2016, 9:48 a.m., Guangya Liu wrote: > > 3rdparty/stout/include/stout/mac.hpp, line 79 > > <https://reviews.apache.org/r/49122/diff/2/?file=1428152#file1428152line79> > > > > What about > > > > Invalid format of MAC address '<MAC>', should be six groups of two > > hexadecimal digits as xx:xx:xx:xx:xx:xx > > > > Including the MAC address in the error msg can show more hints to end > > user.
The caller should print the mac address. - Jie ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49122/#review139228 ----------------------------------------------------------- On June 22, 2016, 11:56 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49122/ > ----------------------------------------------------------- > > (Updated June 22, 2016, 11:56 p.m.) > > > Review request for mesos, Avinash sridharan and Gilbert Song. > > > Repository: mesos > > > Description > ------- > > Added a MAC parse function to stout. > > > Diffs > ----- > > 3rdparty/stout/include/stout/mac.hpp > 1aaafe1380c5300225c2ac9276e87962bd5a0d88 > 3rdparty/stout/tests/mac_tests.cpp f905d4528040ce97d80010b2ba3f13870dca0c7c > > Diff: https://reviews.apache.org/r/49122/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Jie Yu > >
