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

Benjamin Mahler commented on MESOS-994:
---------------------------------------

Those are possibly to deal with the poor semantics of os::getenv (by default it 
crashes if not set, or if passing 'expect=false' then it returns an empty 
string, but this means you can't differentiate between being unset and set to 
empty string!) They should all be going through {{os::getenv}} once it returns 
an Option<string>.

> Add an Option<string> os::getenv() to stout
> -------------------------------------------
>
>                 Key: MESOS-994
>                 URL: https://issues.apache.org/jira/browse/MESOS-994
>             Project: Mesos
>          Issue Type: Improvement
>          Components: stout, technical debt
>            Reporter: Ian Downes
>            Assignee: Greg Mann
>              Labels: newbie
>
> This would replace the common pattern of:
> Option<string> = os::hasenv() ? Option<string>(os::getenv()) : None()



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

Reply via email to