mesos-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Timothy Chen (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (MESOS-2757) Allow mutating T in Option<T>
Date Thu, 21 May 2015 22:37:17 GMT

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

Timothy Chen edited comment on MESOS-2757 at 5/21/15 10:36 PM:
---------------------------------------------------------------

There is already a ticket created: MESOS-2716

And a review board already opened too: https://reviews.apache.org/r/34529/

How about consolidating this into the already opened patch? I see you added a few more operators
as well.


was (Author: tnachen):
There is already a ticket created: MESOS-2716

And a review board already opened too: https://reviews.apache.org/r/34427/

How about consolidating this into the already opened patch? I see you added a few more operators
as well.

> Allow mutating T in Option<T>
> -----------------------------
>
>                 Key: MESOS-2757
>                 URL: https://issues.apache.org/jira/browse/MESOS-2757
>             Project: Mesos
>          Issue Type: Improvement
>          Components: stout
>            Reporter: Joris Van Remoortere
>            Assignee: Joris Van Remoortere
>              Labels: c++11, option, stout
>
> Let's add some operator overloads to Option<T> to allow modification of the T in
Option<T>. This way we can avoid code like:
> {code:title=decoder.hpp|borderStyle=solid}
> http::Pipe::Writer writer_ = writer.get(); // Remove const.
> {code}



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

Mime
View raw message