----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2663/#review3222 -----------------------------------------------------------
src/tests/process_spawn.cpp <https://reviews.apache.org/r/2663/#comment7174> For posterity: we've tried to keep the ordering consistent in the code base. C headers C++ headers Project headers (i.e., those that use "" instead of <>). - Benjamin On 2011-11-02 23:16:55, Charles Reiss wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2663/ > ----------------------------------------------------------- > > (Updated 2011-11-02 23:16:55) > > > Review request for mesos. > > > Summary > ------- > > Currently, tests/process_spawn.cpp uses INT_MAX but only includes <iostream> > which is not gaurenteed (and apparently does not on some systems) to define > INT_MAX, causing a compilation error. > > > This addresses bug MESOS-37. > https://issues.apache.org/jira/browse/MESOS-37 > > > Diffs > ----- > > src/tests/process_spawn.cpp 4f60296 > > Diff: https://reviews.apache.org/r/2663/diff > > > Testing > ------- > > > Thanks, > > Charles > >
