----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35494/#review88181 -----------------------------------------------------------
Patch looks great! Reviews applied: [35494] All tests passed. - Mesos ReviewBot On June 16, 2015, 11:27 a.m., Alexander Rojas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35494/ > ----------------------------------------------------------- > > (Updated June 16, 2015, 11:27 a.m.) > > > Review request for mesos, Bernd Mathiske and Joerg Schad. > > > Bugs: MESOS-2705 > https://issues.apache.org/jira/browse/MESOS-2705 > > > Repository: mesos > > > Description > ------- > > The general rule to format templates is to declare them as: > > ```c++ > template <typename T> // notice the space between template and < > class Foo { > … > }; > ``` > > However, the style is not documented anywhere nor it is inherited from the > Google style guide. > > > Diffs > ----- > > docs/mesos-c++-style-guide.md 94107ed21c6f09349ce691f9f4d36b43bbbe809e > > Diff: https://reviews.apache.org/r/35494/diff/ > > > Testing > ------- > > > Thanks, > > Alexander Rojas > >
