----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17341/#review34926 -----------------------------------------------------------
Ship it! In addition to the 'socket' follow up can you check if we can kill any includes from process.cpp? - Benjamin Hindman 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 > >
