You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@mesos.apache.org by "Benjamin Mahler (JIRA)" <ji...@apache.org> on 2018/01/16 19:54:00 UTC
[jira] [Commented] (MESOS-2921) Add move constructors / assignment
to Result.
[ 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 <ma...@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)