Rust 1.35.0付属のrustfmt 1.2.0-stableとclippy 0.0.212に対応

[yuezato]

このPRの狙い

Rust 1.35.0にバージョンが上がり、clippyのバージョンが以下のようになりました

$ cargo clippy --version
clippy 0.0.212 (265318db 2019-05-17)

これに伴い以下のようなwarningが生じています:

warning: identical conversion
   --> src/storage/journal/region.rs:333:23
    |
333 |         for result in track!(self.ring_buffer.restore_entries())? {
    |                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider removing `track!(self.ring_buffer.restore_entries())?()`: `track!(self.ring_buffer.restore_entries())?`
    |
    = note: #[warn(clippy::identity_conversion)] on by default
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#identity_conversion
warning: identical conversion
   --> src/storage/journal/region.rs:333:23
    |
333 |         for result in track!(self.ring_buffer.restore_entries())? {
    |                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider removing `track!(self.ring_buffer.restore_entries())?()`: `track!(self.ring_buffer.restore_entries())?`
    |
    = note: #[warn(clippy::identity_conversion)] on by default
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#identity_conversion

これを解決します。

内容

解決するといってもtrack!を取り除くことはできないので、clippy::identity_conversionをallowしました。

追加の変更点

Rust fmtがしばらく実行されていなかったようなので、これも併せて実行します。

rustfmt 1.2.0-stable (09940a70 2019-03-27)

[koba-e964]

問題の箇所を以下の2行に分けても (allow(clippy::identity_conversion) なしで) 警告が消えるように見えます:

        let it = track!(self.ring_buffer.restore_entries())?;
        for result in it {

以下の2行だとit?のところで警告が出ます:

        let it = track!(self.ring_buffer.restore_entries());
        for result in it? {

warning: identical conversion
   --> src/storage/journal/region.rs:335:23
    |
335 |         for result in it? {
    |                       ^^^ help: consider removing `it?()`: `it?`
    |
    = note: #[warn(clippy::identity_conversion)] on by default
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#identity_conversion

もしかしたら clippy の偽陽性かもしれません。調べてみます。

[koba-e964]

関連する clippy のコード:
https://github.com/rust-lang/rust-clippy/blob/7b501f0f6af678aa0a081ea1c32d3afa7d1bd641/clippy_lints/src/identity_conversion.rs#L100-L101
Issue: rust-lang/rust-clippy#3913

[koba-e964]

clippy のバグのようなので、いずれ元に戻したいです。
当面は、以下の2種類のうちどちらでも良いような気がします:

• #[allow(clippy::identity_conversion)] を付ける
• 2行に分けて #[allow(clippy::identity_conversion)] を付けない
  いずれ戻す旨のコメントも付けていただけると幸いです。

[koba-e964]

これが必要なのは内部の for 式だけなので、for 式に直接つけてしまってもよい気がしました。

[yuezato]

確かにInner attributeにした方が良いですね。
attributeをつけつつ、後々で外すというコメントも記載する対応でいきたいと思います。

[koba-e964]

LGTM です!