mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alex Clemmer" <>
Subject Re: Review Request 39201: Included `stout/check.hpp` in `future.hpp`.
Date Tue, 13 Oct 2015 18:28:09 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Oct. 13, 2015, 6:28 p.m.)

Review request for mesos, Artem Harutyunyan, Joris Van Remoortere, and Joseph Wu.

Repository: mesos


The Windows integration work will require us to support `stout/net.hpp`,
but this file includes `stout/os.hpp`, which would be onerous to port.

To avoid having to port that entire file, we simply roped in what we
needed from `stout/net.hpp`. A consequence of unincluding os.hpp is that
`process/future.hpp` now requires a direct inclusion of

Diffs (updated)

  3rdparty/libprocess/include/process/future.hpp 9006b8a83d03eab6e67de12a954110029b7d150e




Alex Clemmer

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message