----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62698/ -----------------------------------------------------------
(Updated Sept. 29, 2017, 10:18 p.m.) Review request for mesos, Benjamin Hindman and Vinod Kone. Summary (updated) ----------------- Fixed the clang complation problem for the executor tests. Repository: mesos Description (updated) ------- It appears that g++ throws away the cv-qualifiers when doing the lvalue-to-rvalue conversion for lambdas returning strings but clang does not, so we make `f3` in `THREADSAFE_Executor_Execute` return a non-constant string. Diffs (updated) ----- 3rdparty/libprocess/src/tests/process_tests.cpp e49431873393e835bf0e0a9c240e5d54f06e0ab7 Diff: https://reviews.apache.org/r/62698/diff/2/ Changes: https://reviews.apache.org/r/62698/diff/1-2/ Testing ------- make check with gcc 5.4.0 and clang 3.4.2. Thanks, Chun-Hung Hsiao