> On Feb. 14, 2014, 12:59 a.m., Ben Mahler wrote: > > 3rdparty/libprocess/src/http.cpp, lines 1-8 > > <https://reviews.apache.org/r/17341/diff/3/?file=484624#file484624line1> > > > > Looks like we could add some includes here: > > > > #include <cstring> > > #include <iostream> > > #include <string> > > > > #include <process/future.hpp> > > > > #include <stout/lambda.hpp> > > #include <stout/nothing.hpp> > > #include <stout/option.hpp> > > #include <stout/os.hpp> > > #include <stout/try.hpp>
just added them - Charlie ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17341/#review34447 ----------------------------------------------------------- On Feb. 19, 2014, 8:18 p.m., Charlie Carson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17341/ > ----------------------------------------------------------- > > (Updated Feb. 19, 2014, 8:18 p.m.) > > > Review request for mesos, Benjamin Hindman and Jie Yu. > > > Bugs: https://issues.apache.org/jira/browse/MESOS-936 > > https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/MESOS-936 > > > Repository: mesos-git > > > Description > ------- > > move process::http namespace into http.cpp > > This moves the code declared in the process::http namespace to > be in it's own file (http.cpp). It already had it's own header. > > SEE: https://issues.apache.org/jira/browse/MESOS-936 > > Review: https://reviews.apache.org/r/17341 > > > Diffs > ----- > > 3rdparty/libprocess/Makefile.am bcb78a45788fe93c520a787c2b614d49c354e633 > 3rdparty/libprocess/src/http.cpp PRE-CREATION > 3rdparty/libprocess/src/process.cpp > 7bf2d70c59455f52b26374f78198c5c7dc0a883b > > Diff: https://reviews.apache.org/r/17341/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Charlie Carson > >
