----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60057/ -----------------------------------------------------------
Review request for mesos and Michael Park. Repository: mesos Description ------- This member function of 'ProcessManager' was capturing a 'this' pointer when dispatching to itself, but did not properly use 'defer' or 'dispatch'. While this pattern is usually suspect, it was safe here as we can be sure that the process manager lives long enough to safely invoke the created callback. This patch removes the capture of 'this' and instead explicitly references the 'static process_manager' in the created callback to signal that we rely on external invariants. Diffs ----- 3rdparty/libprocess/src/process.cpp 4ff7448d171f39dbb8cbb81dd9bed136ad43d62d Diff: https://reviews.apache.org/r/60057/diff/1/ Testing ------- * `make check` (Fedora 25 clang-trunk w/ optimizations) Thanks, Benjamin Bannier
