needless-question-mark

Disallow propagation immediately wrapped in the same Result type

Propagating a Result payload and immediately rebuilding an identical successful Result performs two redundant operations.
Instead, you SHOULD return the original Result directly.

- Category: style
- Level: warning
- Fix: automatic
- Scope: module

Reported

function forward(result: Result<int32, string>): Result<int32, string> {
    return Result.ok(result?);
}
Accepted

function forward(result: Result<int32, string>): Result<int32, string> {
    return result;
}
Prior art

- Clippy · needlessquestionmark

language/linter/src/rules/style/needlessquestionmark.rs:8