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)