You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@arrow.apache.org by "Neville Dipale (Jira)" <ji...@apache.org> on 2020/09/29 23:52:00 UTC

[jira] [Commented] (ARROW-10108) [Rust] [Parquet] Fix compiler warning about unused return value

    [ https://issues.apache.org/jira/browse/ARROW-10108?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17204366#comment-17204366 ] 

Neville Dipale commented on ARROW-10108:
----------------------------------------

There are a lot of clippy issues with the latest nightly too, perhaps we could move to the latest nightly after the 2.0.0 release? I'd also like for us to perform destructive refactors soon as possible after the release (rearranging the arrow::array module by splitting arrays into their own files)

> [Rust] [Parquet] Fix compiler warning about unused return value
> ---------------------------------------------------------------
>
>                 Key: ARROW-10108
>                 URL: https://issues.apache.org/jira/browse/ARROW-10108
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: Rust
>            Reporter: Andy Grove
>            Priority: Major
>             Fix For: 2.0.0
>
>
> When compiling with latest nightly, this warning is produced:
> {code:java}
> warning: unused return value of `std::mem::replace` that must be used
>    --> parquet/src/encodings/encoding.rs:391:9
>     |
> 391 |         mem::replace(&mut self.hash_slots, new_hash_slots);
>     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>     |
>     = note: `#[warn(unused_must_use)]` on by default
>     = note: if you don't need the old value, you can just assign the new value directly {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)