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

Benjamin Mahler commented on MESOS-2921:
----------------------------------------

{noformat}
commit a6b3d1ad6f4e1b83b48ac58ba247a422fac32101
Author: Anand Mazumdar <mazumdar.an...@gmail.com>
Date:   Tue May 31 13:28:43 2016 -0600

    Added move constructor/assignment operator to `Result`.

    Added move constructor/assignment operator to \`Result\`.
    Note that `Some` still makes a copy and would be fixed in
    a separate patch.

    Review: https://reviews.apache.org/r/47986/
{noformat}

[~mcypark] before closing this, looks like we also need to add a default one 
for {{Result&&}}, similar to what was done for Try?
https://github.com/apache/mesos/commit/9942fa7749aaee6187b9f565511ed1fdd1c7c26c

> Add move constructors / assignment to Result.
> ---------------------------------------------
>
>                 Key: MESOS-2921
>                 URL: https://issues.apache.org/jira/browse/MESOS-2921
>             Project: Mesos
>          Issue Type: Improvement
>          Components: stout
>            Reporter: Benjamin Mahler
>            Priority: Major
>
> Now that we have C++11, let's add move constructors and move assignment 
> operators for Result, similarly to what was done for Option.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to