Repository navigation
The suggestion to remove generic parameters proposed by help did not remove comma #128302
Description
Code
type Result<T> = std::result::Result<T, ()>;
fn foo() -> Result<'static, ()> { Ok(()) }
fn main() { }Current output
error[E0107]: type alias takes 0 lifetime arguments but 1 lifetime argument was supplied
--> src/main.rs:3:13
|
3 | fn foo() -> Result<'static, ()> { Ok(()) }
| ^^^^^^ ------- help: remove this lifetime argument
| |
| expected 0 lifetime arguments
|
note: type alias defined here, with 0 lifetime parameters
--> src/main.rs:1:6
|
1 | type Result<T> = std::result::Result<T, ()>;
| ^^^^^^
For more information about this error, try `rustc --explain E0107`.
error: could not compile `test2` (bin "test2") due to 1 previous errorDesired output
error[E0107]: type alias takes 0 lifetime arguments but 1 lifetime argument was supplied
--> src/main.rs:3:13
|
3 | fn foo() -> Result<'static, ()> { Ok(()) }
| ^^^^^^ --------- help: remove this lifetime argument
| |
| expected 0 lifetime arguments
|
note: type alias defined here, with 0 lifetime parameters
--> src/main.rs:1:6
|
1 | type Result<T> = std::result::Result<T, ()>;
| ^^^^^^
For more information about this error, try `rustc --explain E0107`.
error: could not compile `test2` (bin "test2") due to 1 previous errorRationale and extra context
Due to not removing the comma, rust-analyzer still has an error after quick fix
Other cases
No response
Rust Version
rustc 1.80.0 (051478957 2024-07-21)
binary: rustc
commit-hash: 051478957371ee0084a7c0913941d2a8c4757bb9
commit-date: 2024-07-21
host: aarch64-unknown-linux-gnu
release: 1.80.0
LLVM version: 18.1.7Anything else?
No response
Activity
tgross35 commentedon Jul 28, 2024
It looks like that may have been fixed by b30fdec, can you check what happens on the latest nightly? (Unfortunately the playground hasn't been updated to something recent enough yet).
A4-Tacks commentedon Jul 28, 2024
rustc 1.82.0-nightly (7c2012d 2024-07-26)
The cargo check output doesn't seem to have changed
A4-Tacks commentedon Jul 28, 2024
In the example of b30fdec, it seems that the test case does not include a comma that removes the first parameter, but rather the tail parameter
tgross35 commentedon Jul 28, 2024
Cc @estebank since you did #127374, might be a quick fixup
A4-Tacks commentedon Jul 28, 2024
This doesn't seem to remove the comma on the left either
output:
chansuke commentedon Aug 12, 2024
@rustbot claim