rust-lang/rust-clippy

Two suggestions for more DRY code

Open

#1,674 建立於 2017年4月14日

在 GitHub 查看
 (3 留言) (0 反應) (0 負責人)Rust (10,406 star) (1,391 fork)batch import
A-lintL-styleT-middlegood first issue

描述

Given this code:

struct Foo { cap: usize }
impl Foo {
    fn new() -> Foo { Foo { cap: 1 } }
    fn with_capacity(cap: usize) -> Foo {
        Foo { cap: cap }
    }
}
fn main() {}

In my opinion Clippy should suggest to replace four of those usages of "Foo" with "Self" and to remove one ": cap", making the code more DRY, the intended resulting code should be:

struct Foo { cap: usize }
impl Foo {
    fn new() -> Self { Self { cap: 1 } }
    fn with_capacity(cap: usize) -> Self {
        Self { cap }
    }
}
fn main() {}

貢獻者指南