You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@mesos.apache.org by "Yan Xu (JIRA)" <ji...@apache.org> on 2017/03/06 23:30:33 UTC

[jira] [Comment Edited] (MESOS-7195) Use C++11 variadic templates for process::dispatch/defer/delay/async/run

    [ https://issues.apache.org/jira/browse/MESOS-7195?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15898378#comment-15898378 ] 

Yan Xu edited comment on MESOS-7195 at 3/6/17 11:30 PM:
--------------------------------------------------------

[~mcypark] I am thinking of investigating this. Just would like to solicit some feedback first to help me get started.

In the slack channel you mentioned:
{quote}
it’ll be some work to implement the variadic template versions
because it’ll involve some API changes
{quote}

Could you elaborate a bit further (plus other suggestions)?


was (Author: xujyan):
[~mcypark] I am thinking of investigating this. Just would like to solicit some feedback first to help me get started.

In the slack channel you mentioned:
{quote}
it’ll be some work to implement the variadic template versions
because it’ll involve some API changes
{quote}

Could you elaborate a bit further?

> Use C++11 variadic templates for process::dispatch/defer/delay/async/run
> ------------------------------------------------------------------------
>
>                 Key: MESOS-7195
>                 URL: https://issues.apache.org/jira/browse/MESOS-7195
>             Project: Mesos
>          Issue Type: Improvement
>          Components: libprocess
>            Reporter: Yan Xu
>
> These methods are currently implemented using {{REPEAT_FROM_TO}} (i.e., {{BOOST_PP_REPEAT_FROM_TO}}):
> {code:title=}
> REPEAT_FROM_TO(1, 11, TEMPLATE, _) // Args A0 -> A9.
> {code}
> This means we have to bump up the number of repetition whenever we have a new method with more args.
> Seems like we can replace this with C++11 variadic templates.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)