You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@arrow.apache.org by "Andrew Lamb (Jira)" <ji...@apache.org> on 2021/03/19 10:39:00 UTC
[jira] [Updated] (ARROW-12024) [Rust] Lint failuers in
arrow/src/array/array_binary.rs:427:20
[ https://issues.apache.org/jira/browse/ARROW-12024?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Andrew Lamb updated ARROW-12024:
--------------------------------
Summary: [Rust] Lint failuers in arrow/src/array/array_binary.rs:427:20 (was: [Rust] Rust 1.52 has additional clippy lint failure)
> [Rust] Lint failuers in arrow/src/array/array_binary.rs:427:20
> --------------------------------------------------------------
>
> Key: ARROW-12024
> URL: https://issues.apache.org/jira/browse/ARROW-12024
> Project: Apache Arrow
> Issue Type: Bug
> Components: Rust
> Reporter: Andrew Lamb
> Assignee: Andrew Lamb
> Priority: Major
>
> Master has some lint failures:
> https://github.com/apache/arrow/pull/9749/checks?check_run_id=2144048180
> {code}
> error: this `else { if .. }` block can be collapsed
> --> arrow/src/array/array_binary.rs:427:20
> |
> 427 | } else {
> | ____________________^
> 428 | | if let Some(size) = size {
> 429 | | buffer.extend_zeros(size);
> 430 | | } else {
> 431 | | prepend += 1;
> 432 | | }
> 433 | | }
> | |_____________^
> |
> = note: `-D clippy::collapsible-if` implied by `-D warnings`
> = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_if
> help: collapse nested if block
> |
> 427 | } else if let Some(size) = size {
> 428 | buffer.extend_zeros(size);
> 429 | } else {
> 430 | prepend += 1;
> 431 | }
> |
> {code}
> Reproduce via running `rustup`
--
This message was sent by Atlassian Jira
(v8.3.4#803005)