[jira] [Comment Edited] (MESOS-2852) Remove now that C++11 is required.

["Michael Park (JIRA)" <ji...@apache.org>]

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

Michael Park edited comment on MESOS-2852 at 6/14/15 5:04 PM:
--------------------------------------------------------------

The reason we kept {{<stout/lambda.hpp>}} is because of the placeholders. [~benjaminhindman] mentioned that the {{lambda::_1}} is probably preferable to {{std::placeholders::_1}}, or other alternatives which would likely to involve using-namespace declarations.

  (1) Keep the {{lambda::}}
  (2) Drop {{lambda::}} and be ok with using {{std::placeholders}}
  (3) Drop {{lambda::} for all but placeholders

Do you have a preference between the 3 or perhaps other suggestions?


was (Author: mcypark):
The reason we kept {{<stout/lambda.hpp>}} is because of the placeholders. [~benjaminhindman] mentioned that the {{lambda::_1}} is probably preferable to {{std::placeholders::_1}}, or other alternatives which probably involves using-namespace declarations.

  (1) Keep the {{lambda::}}
  (2) Drop {{lambda::}} and be ok with using {{std::placeholders}}
  (3) Drop {{lambda::} for all but placeholders

Do you have a preference between the 3 or perhaps other suggestions?

> Remove <stout/lambda.hpp> now that C++11 is required.
> -----------------------------------------------------
>
>                 Key: MESOS-2852
>                 URL: https://issues.apache.org/jira/browse/MESOS-2852
>             Project: Mesos
>          Issue Type: Task
>          Components: stout, technical debt
>            Reporter: Benjamin Mahler
>              Labels: mesosphere
>
> Now that we no longer need to pull in tr1, let's remove <stout/lambda.hpp>. It looks like std::function has already crept its way into the libprocess source, but the mesos source still uses {{lambda::}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)