[jira] [Resolved] (ARROW-7395) [C++] Logical "or" with constants is a Clang warning

["Antoine Pitrou (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/ARROW-7395?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Antoine Pitrou resolved ARROW-7395.
-----------------------------------
    Fix Version/s: 1.0.0
       Resolution: Fixed

Issue resolved by pull request 6036
[https://github.com/apache/arrow/pull/6036]

> [C++] Logical "or" with constants is a Clang warning
> ----------------------------------------------------
>
>                 Key: ARROW-7395
>                 URL: https://issues.apache.org/jira/browse/ARROW-7395
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: C++
>    Affects Versions: 0.15.0
>            Reporter: Jim Apple
>            Assignee: Jim Apple
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.0.0
>
>          Time Spent: 50m
>  Remaining Estimate: 0h
>
> With clang version 9.0.1, the C++ debug build fails with:
> {noformat}
> In file included from /home/jbapple/code/arrow/cpp/src/arrow/vendored/xxhash/xxhash.h:532:
> /home/jbapple/code/arrow/cpp/src/arrow/vendored/xxhash/xxhash.c:810:11: error: use of logical '||' with constant operand [-Werror,-Wconstant-logical-operand]
>     if (0 || 0) {
>           ^  ~
> /home/jbapple/code/arrow/cpp/src/arrow/vendored/xxhash/xxhash.c:810:11: note: use '|' for a bitwise operation
>     if (0 || 0) {
>           ^~
>           |
> {noformat}
> The simple fix is to add {{-Wno-constant-logical-operand}} to SetupCxxFlags.cmake.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)