----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73137/#review222464 -----------------------------------------------------------
Ship it! 3rdparty/libprocess/src/posix/libev/libev.cpp Lines 58 (patched) <https://reviews.apache.org/r/73137/#comment311538> May be worth providing a default `false` value for `triggered` instead of relying on `Latch` alawys being created using value initialization. 3rdparty/libprocess/src/posix/libev/libev.cpp Lines 63 (patched) <https://reviews.apache.org/r/73137/#comment311539> Move this closer to the `loop` declaration? 3rdparty/libprocess/src/posix/libev/libev.cpp Lines 200 (patched) <https://reviews.apache.org/r/73137/#comment311540> Seems unnecessary. - Ilya Pronin On Jan. 12, 2021, 11:22 a.m., Benjamin Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73137/ > ----------------------------------------------------------- > > (Updated Jan. 12, 2021, 11:22 a.m.) > > > Review request for mesos and Ilya Pronin. > > > Repository: mesos > > > Description > ------- > > The default loop in libev supports child process events, which we > don't want and explicitly ignore. We can just use dynamic loops > instead which allows us to properly destroy the loop when libprocess > finalizes. > > This also helps minimize the changes needed once we add support for > multiple loops, since those need to be dynamic loops as well. > > > Diffs > ----- > > 3rdparty/libprocess/src/posix/libev/libev.cpp > b38e7a0f882a8c24950bdc6fd74a4d25fc68549e > > > Diff: https://reviews.apache.org/r/73137/diff/1/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Mahler > >
