> On Feb. 6, 2015, 3:44 p.m., Kapil Arya wrote: > > src/tests/module.hpp, line 58 > > <https://reviews.apache.org/r/30728/diff/1/?file=852159#file852159line58> > > > > Do we have it in our style sheet to not use two spaces when followed by > > a period. Some people prefer two spaces to one space :-).
I updated this change before committing so that it only removes the extra declaration of cgroups::cpu::shares. - Ben ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30728/#review71441 ----------------------------------------------------------- On Feb. 6, 2015, 2:59 p.m., Alexander Rukletsov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30728/ > ----------------------------------------------------------- > > (Updated Feb. 6, 2015, 2:59 p.m.) > > > Review request for mesos, Ben Mahler, Kapil Arya, and Timothy Chen. > > > Repository: mesos > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/linux/cgroups.hpp abf31df1b4dbf6f715f93256b83c9996a45099cf > src/tests/module.hpp 776a4263627492f1a7b360a54c83ef34d4511ae1 > > Diff: https://reviews.apache.org/r/30728/diff/ > > > Testing > ------- > > Make check (OS X 10.9.5) > > > Thanks, > > Alexander Rukletsov > >
