[ARROW-12864] [C++] Remove needless out argument from arrow::internal::InvertBitmap

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 4.0.0
• Fix Version/s: 5.0.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/28596
• Assignee: Niranda Perera
• Reporter: Niranda Perera
• Created: 25/May/21 00:57
• Updated: 11/Jan/23 08:29
• Resolved: 25/May/21 01:55

Description

arrow::internal::InvertBitmap is defined in bitmap_ops.h

but there seems to be a small mistake in the impl bimap_ops.cc [2] (more like a type IMO)

[1] https://github.com/apache/arrow/blob/master/cpp/src/arrow/util/bitmap_ops.h#L81

[2] https://github.com/apache/arrow/blob/master/cpp/src/arrow/util/bitmap_ops.cc#L392

Issue Links

• links to: GitHub Pull Request #10391