----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26789/#review56988 -----------------------------------------------------------
Ship it! I'll fix the last nits. src/module/manager.hpp <https://reviews.apache.org/r/26789/#comment97426> s/ / / src/module/manager.cpp <https://reviews.apache.org/r/26789/#comment97427> s/> >/>>/ src/module/manager.cpp <https://reviews.apache.org/r/26789/#comment97428> s/ / / - Niklas Nielsen On Oct. 16, 2014, 11:43 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, 11:43 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 > >
