You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@arrow.apache.org by "Wes McKinney (Jira)" <ji...@apache.org> on 2021/02/20 20:42:00 UTC
[jira] [Commented] (ARROW-6399) [C++] More extensive attributes
usage could improve debugging
[ 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)