You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@mesos.apache.org by Kevin Klues <kl...@gmail.com> on 2016/01/29 11:09:47 UTC

Review Request 42956: Added variadic template for process::await.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42956/
-----------------------------------------------------------

Review request for mesos, Joris Van Remoortere and Michael Park.


Bugs: MESOS-4551
    https://issues.apache.org/jira/browse/MESOS-4551


Repository: mesos


Description
-------

Previously, templates only existed to allow await to take either a
std::list of Futures, *exactly* 2 Futures as arguments, or *exactly* 3
Futures as arguments.

This commit removes the 2 and 3 argument templates and replaces them
with a variadic template that accepts an arbitrary number of Futures as
arguments.


Diffs
-----

  3rdparty/libprocess/include/process/collect.hpp cd78b6c211c2e4ca2b1ebbe728cfc2dfad1a32c9 

Diff: https://reviews.apache.org/r/42956/diff/


Testing
-------


Thanks,

Kevin Klues