----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26789/#review56893 -----------------------------------------------------------
Bad patch! Reviews applied: [26789] Failed command: git apply --index 26789.patch Error: error: patch failed: src/module/manager.hpp:99 error: src/module/manager.hpp: patch does not apply error: patch failed: src/module/manager.cpp:180 error: src/module/manager.cpp: patch does not apply error: patch failed: src/tests/module_tests.cpp:134 error: src/tests/module_tests.cpp: patch does not apply - Mesos ReviewBot On Oct. 16, 2014, 1:19 a.m., Kapil Arya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26789/ > ----------------------------------------------------------- > > (Updated Oct. 16, 2014, 1:19 a.m.) > > > Review request for mesos, Niklas Nielsen and Till Toenshoff. > > > Repository: mesos-git > > > Description > ------- > > Also removed the existing unloadAll() that is no longer needed. > > unloadAll() forces destruction of Owned pointers to the dynamic library > handle. This implicitly causes a dlclose() of all the loaded module > libraries. However, in certain situations (e.g. OS X), dlclose() is not > ideal due to its side-effects. > > > Diffs > ----- > > src/module/manager.hpp 797728a8c8e88dd1a13142a355cbe0b1491fb7a2 > src/module/manager.cpp 8cc79956a8d3d7cb27016889ec59d138a0b58e45 > src/tests/module_tests.cpp 71e7ef9ee9b21305528c00c7bfa2419ac9936974 > > Diff: https://reviews.apache.org/r/26789/diff/ > > > Testing > ------- > > make check. > > > Thanks, > > Kapil Arya > >
