----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13879/#review25658 -----------------------------------------------------------
Ship it! Ship It! - Vinod Kone On Aug. 28, 2013, 2:30 a.m., Vinson Lee wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13879/ > ----------------------------------------------------------- > > (Updated Aug. 28, 2013, 2:30 a.m.) > > > Review request for mesos, Benjamin Hindman, Ben Mahler, Chi Zhang, Ian > Downes, David Mackey, and Vinod Kone. > > > Repository: mesos-git > > > Description > ------- > > Author: Vinson Lee <[email protected]> > > libprocess: Fix clang mismatched-tags errors. > > This patch fixes these build errors 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: > ../3rdparty/libprocess/include/process/event.hpp:15:1: error: struct > 'ProcessBase' was previously declared as a class [-Werror,-Wmismatched-tags] > struct ProcessBase; > ^ > ../3rdparty/libprocess/include/process/pid.hpp:14:7: note: previous use is > here > class ProcessBase; > ^ > 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: > ../3rdparty/libprocess/include/process/process.hpp:26:1: error: 'ProcessBase' > defined as a class here but previously declared as a struct > [-Werror,-Wmismatched-tags] > class ProcessBase : public EventVisitor > ^ > ../3rdparty/libprocess/include/process/event.hpp:15:1: note: did you mean > class here? > struct ProcessBase; > ^~~~~~ > class > > Signed-off-by: Vinson Lee <[email protected]> > > > Diffs > ----- > > 3rdparty/libprocess/include/process/event.hpp 84a8790 > > Diff: https://reviews.apache.org/r/13879/diff/ > > > Testing > ------- > > > Thanks, > > Vinson Lee > >
