> On Dec. 5, 2014, 6:56 p.m., Niklas Nielsen wrote: > > src/module/manager.cpp, lines 199-200 > > <https://reviews.apache.org/r/28764/diff/1/?file=783851#file783851line199> > > > > I'd prefer not to align around '(' - can you bring both lines down (4 > > space indent)?
Actually just saw that Kapil did it the way you describe after I upped this RR - my bad :) . - Till ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28764/#review64057 ----------------------------------------------------------- On Dec. 5, 2014, 6:48 p.m., Till Toenshoff wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28764/ > ----------------------------------------------------------- > > (Updated Dec. 5, 2014, 6:48 p.m.) > > > Review request for mesos, Kapil Arya and Niklas Nielsen. > > > Repository: mesos-git > > > Description > ------- > > see summary > > > Diffs > ----- > > src/module/manager.cpp b15b0fc > > Diff: https://reviews.apache.org/r/28764/diff/ > > > Testing > ------- > > make check and functional testing with a broken module library. > > > Thanks, > > Till Toenshoff > >
