> On Sept. 26, 2016, 8:43 p.m., Joseph Wu wrote: > > 3rdparty/stout/include/stout/os/windows/read.hpp, line 35 > > <https://reviews.apache.org/r/52196/diff/1/?file=1509013#file1509013line35> > > > > You wrapped the `::recv` in an earlier patch. Want to use it here? > > (`net::recv`)
Good catch (bad merge?). recv() takes the lengths as `int` while _read() takes it as `unsigned int`. - Daniel ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52196/#review150447 ----------------------------------------------------------- On Sept. 23, 2016, 5:05 a.m., Daniel Pravat wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52196/ > ----------------------------------------------------------- > > (Updated Sept. 23, 2016, 5:05 a.m.) > > > Review request for mesos, Alex Clemmer and Joseph Wu. > > > Repository: mesos > > > Description > ------- > > Fixed warnings in `read/write` functions. > > > Diffs > ----- > > 3rdparty/stout/include/stout/os/windows/read.hpp > cb0abf70307f0dbba0b8f68e884df199b0359186 > 3rdparty/stout/include/stout/os/windows/sendfile.hpp > 4c5178e1f3696906aa1cf93b8ca450f61ba79355 > 3rdparty/stout/include/stout/os/windows/write.hpp > 705ad03ee58f9cc822ec1ed25fd10f39059717d4 > > Diff: https://reviews.apache.org/r/52196/diff/ > > > Testing > ------- > > OSX: make check > Windows: build > > > Thanks, > > Daniel Pravat > >
