[ https://issues.apache.org/jira/browse/ARROW-6399?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17287794#comment-17287794 ]
Wes McKinney commented on ARROW-6399: ------------------------------------- Is it possible to go ahead and add {{ARROW_MUST_USE_RESULT}} to {{Result<T>}}? > [C++] More extensive attributes usage could improve debugging > ------------------------------------------------------------- > > Key: ARROW-6399 > URL: https://issues.apache.org/jira/browse/ARROW-6399 > Project: Apache Arrow > Issue Type: Improvement > Components: C++ > Reporter: Ben Kietzman > Priority: Minor > > Wrapping raw or smart pointer parameters and other declarations with > {{gsl::not_null}} will assert they are not null. The check is dropped for > release builds. > Status is tagged with ARROW_MUST_USE_RESULT which emits warnings when a > Status might be ignored if compiling with clang, but Result<> should probably > be tagged with this too -- This message was sent by Atlassian Jira (v8.3.4#803005)