----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13840/#review25776 -----------------------------------------------------------
Ship it! Sorry for the run-around here, I've committed this with a small change to fix the include order. Please mark as submitted, thanks Vinson! - Ben Mahler On Aug. 28, 2013, 12:26 a.m., Vinson Lee wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13840/ > ----------------------------------------------------------- > > (Updated Aug. 28, 2013, 12:26 a.m.) > > > Review request for mesos, Benjamin Hindman, Ben Mahler, Chi Zhang, Eric > Biederman, Ian Downes, and David Mackey. > > > Repository: mesos-git > > > Description > ------- > > Author: Vinson Lee <[email protected]> > > stout: Fix clang return-type build error. > > This patch fixes this build error with clang. > > In file included from sched/sched.cpp:35: > In file included from ../3rdparty/libprocess/include/process/delay.hpp:6: > In file included from ../3rdparty/libprocess/include/process/dispatch.hpp:9: > In file included from ../3rdparty/libprocess/include/process/process.hpp:13: > In file included from ../3rdparty/libprocess/include/process/event.hpp:10: > In file included from ../3rdparty/libprocess/include/process/socket.hpp:10: > In file included from > ../3rdparty/libprocess/3rdparty/stout/include/stout/os.hpp:65: > ../3rdparty/libprocess/3rdparty/stout/include/stout/os/sendfile.hpp:46:1: > error: control may reach end of non-void function [-Werror,-Wreturn-type] > } > ^ > > Signed-off-by: Vinson Lee <[email protected]> > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/sendfile.hpp b41ba63 > > Diff: https://reviews.apache.org/r/13840/diff/ > > > Testing > ------- > > > Thanks, > > Vinson Lee > >
