[PARQUET-1843] [C++] Unnecessary assignment in DictDecoderImpl::Decode

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: cpp-4.0.0
• Component/s: parquet-cpp
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: Kazuaki Ishizaki
• Created: 17/Apr/20 01:20
• Updated: 17/Apr/20 03:05
• Resolved: 17/Apr/20 03:05

Description

The following duplicated assignments are unnecessary.

   int DecodeIndices(int num_values, arrow::ArrayBuilder* builder) override {
     num_values = std::min(num_values, num_values_);
     num_values = std::min(num_values, num_values_);
     if (num_values > 0) {

Issue Links

• links to: GitHub Pull Request #6960