> On Jan. 29, 2015, 7:17 p.m., Michael Park wrote: > > src/master/master.cpp, lines 2937-2940 > > <https://reviews.apache.org/r/30283/diff/2/?file=840291#file840291line2937> > > > > Just wondering, how come the braces were removed for the `default` case?
We won't be adding more stuff to the default case. That's the reason I removed the braces:) - Jie ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30283/#review70250 ----------------------------------------------------------- On Jan. 29, 2015, 7:03 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30283/ > ----------------------------------------------------------- > > (Updated Jan. 29, 2015, 7:03 p.m.) > > > Review request for mesos, Ben Mahler and Michael Park. > > > Repository: mesos-git > > > Description > ------- > > To avoid potential conflicts. > > > Diffs > ----- > > src/master/master.cpp 54f26900ac8c63e79a1f89562a988c9a2567d209 > > Diff: https://reviews.apache.org/r/30283/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Jie Yu > >
