You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "Antoine Pitrou (Jira)" <ji...@apache.org> on 2019/09/30 10:11:00 UTC
[jira] [Resolved] (ARROW-6735) [C++] Suppress sign-compare warning
with g++ 9.2.1
[ https://issues.apache.org/jira/browse/ARROW-6735?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Antoine Pitrou resolved ARROW-6735.
-----------------------------------
Fix Version/s: 0.15.0
Resolution: Fixed
Issue resolved by pull request 5541
[https://github.com/apache/arrow/pull/5541]
> [C++] Suppress sign-compare warning with g++ 9.2.1
> --------------------------------------------------
>
> Key: ARROW-6735
> URL: https://issues.apache.org/jira/browse/ARROW-6735
> Project: Apache Arrow
> Issue Type: Improvement
> Components: C++
> Reporter: Kouhei Sutou
> Assignee: Kouhei Sutou
> Priority: Major
> Labels: pull-request-available
> Fix For: 0.15.0
>
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> This was introduced by ARROW-6402 but removed accidentally by ARROW-5935.
> {noformat}
> In file included from ../src/arrow/status.h:24,
> from ../src/arrow/memory_pool.h:26,
> from ../src/arrow/buffer.h:28,
> from ../src/arrow/array.h:28,
> from ../src/arrow/array/builder_union.h:25,
> from ../src/arrow/array/builder_union.cc:18:
> ../src/arrow/array/builder_union.cc: In constructor 'arrow::BasicUnionBuilder::BasicUnionBuilder(arrow::MemoryPool*, arrow::UnionMode::type, const std::vector<std::shared_ptr<arrow::ArrayBuilder> >&, const std::shared_ptr<arrow::DataType>&)':
> ../src/arrow/util/logging.h:86:55: error: comparison of integer expressions of different signedness: 'std::vector<arrow::ArrayBuilder*>::size_type' {aka 'long unsigned int'} and 'signed char' [-Werror=sign-compare]
> 86 | #define ARROW_CHECK_LT(val1, val2) ARROW_CHECK((val1) < (val2))
> | ~~~~~~~^~~~~~~~
> ../src/arrow/util/macros.h:43:52: note: in definition of macro 'ARROW_PREDICT_TRUE'
> 43 | #define ARROW_PREDICT_TRUE(x) (__builtin_expect(!!(x), 1))
> | ^
> ../src/arrow/util/logging.h:86:36: note: in expansion of macro 'ARROW_CHECK'
> 86 | #define ARROW_CHECK_LT(val1, val2) ARROW_CHECK((val1) < (val2))
> | ^~~~~~~~~~~
> ../src/arrow/util/logging.h:135:19: note: in expansion of macro 'ARROW_CHECK_LT'
> 135 | #define DCHECK_LT ARROW_CHECK_LT
> | ^~~~~~~~~~~~~~
> ../src/arrow/array/builder_union.cc:63:3: note: in expansion of macro 'DCHECK_LT'
> 63 | DCHECK_LT(type_id_to_children_.size(), std::numeric_limits<int8_t>::max());
> | ^~~~~~~~~
> {noformat}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)