> On Oct. 22, 2014, 9:23 a.m., Niklas Nielsen wrote: > > src/examples/test_module.hpp, lines 51-52 > > <https://reviews.apache.org/r/26906/diff/7/?file=728685#file728685line51> > > > > Why not just return the string directly? > > > > return "TestModule"
Ah - never mind. It is because you return a pointer and the string needs to reside somewhere :-/ - Niklas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26906/#review57808 ----------------------------------------------------------- On Oct. 22, 2014, 1:04 a.m., Kapil Arya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26906/ > ----------------------------------------------------------- > > (Updated Oct. 22, 2014, 1:04 a.m.) > > > Review request for mesos, Benjamin Hindman and Niklas Nielsen. > > > Repository: mesos-git > > > Description > ------- > > Since, there is no reliable method of converting a template type to a > string, we create an temporary object of the given kind and compare the > "kind" fields of the temporary object with that of the one being > created. > > > Diffs > ----- > > include/mesos/module.hpp 5205e30c7b8228eced9793aef2348930180811bb > src/examples/test_module.hpp b5cc2fa9147c4fe9a1f2eb3e8c2c2565ea7f7420 > src/module/manager.hpp 1c93ac4707e3c5dc8acaaa23861b62d3d3921125 > src/tests/module_tests.cpp e92d8717427e259b1fa26b3f17719fd616a748c7 > > Diff: https://reviews.apache.org/r/26906/diff/ > > > Testing > ------- > > Added a new test and ran make check. > > > Thanks, > > Kapil Arya > >
