rust-lang/rust-clippy

lint idea: `useless_default_generic_parameters`

Open

#14 848 ouverte le 19 mai 2025

Voir sur GitHub
 (12 commentaires) (3 réactions) (1 assigné)Rust (1 391 forks)batch import
A-lintG-Rust-for-Linuxgood first issue

Métriques du dépôt

Stars
 (10 406 stars)
Métriques de merge PR
 (Merge moyen 16j 6h) (79 PRs mergées en 30 j)

Description

What it does

Given a type with a generic argument that has a default:

type Result<T = ()> = core::result::Result<T, MyError>;

Usage of this type when specifying the default should not use generics:

fn foo() -> Result<()> { Ok(()) }
//                ^^^^ unnecessary generic, `()` already is the default
//                hint: use `Result` instead.

Advantage

  • Removes duplication of the default value
  • Reduces visual clutter
  • Reminds people that the type has a default value

Drawbacks

  • when a library adds a default value, one gets this warning when updating
  • macros might trigger this involuntarily

Example

type Result<T = ()> = core::result::Result<T, MyError>;

fn foo() -> Result<()> {
    Ok(())
}

Could be written as:

type Result<T = ()> = core::result::Result<T, MyError>;

fn foo() -> Result {
    Ok(())
}

Guide contributeur