----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26789/#review56850 -----------------------------------------------------------
LGTM module my comments src/module/manager.cpp <https://reviews.apache.org/r/26789/#comment97284> "Only open a library once"? Dlopen is hidden by the abstraction :-) src/module/manager.cpp <https://reviews.apache.org/r/26789/#comment97285> Update this comment to reflect the new unload() call - Niklas Nielsen On Oct. 15, 2014, 5:18 p.m., Kapil Arya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26789/ > ----------------------------------------------------------- > > (Updated Oct. 15, 2014, 5:18 p.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 > >
