rust-analyzer icon indicating copy to clipboard operation
rust-analyzer copied to clipboard
verified publisher icon rust-lang

fix: use 4 spaces for indentation in macro expansion

Open roife opened this issue 2 years ago • 2 comments

Partial fix for #16471.

In the previous code, the indentation produced by macro expansion was set to 2 spaces. This PR modifies it to 4 spaces for the sake of consistency.

roife avatar Feb 27 '24 05:02 roife

I don't think we can do much about 1. right now as our search infra is just not built in a way that allows to search like that

Veykril avatar Feb 27 '24 09:02 Veykril

I agree that replacement of self keyword is non-trivial since it may refer the 'current module'. I have changed the title and the description of this PR.

roife avatar Feb 27 '24 12:02 roife

@rustbot ready

roife avatar Mar 04 '24 08:03 roife

Thanks! @bors r+

Veykril avatar Mar 04 '24 08:03 Veykril

:pushpin: Commit 76b86b24bd38e53ee8a3555608339803ecb5cbb9 has been approved by Veykril

It is now in the queue for this repository.

bors avatar Mar 04 '24 08:03 bors

:hourglass: Testing commit 76b86b24bd38e53ee8a3555608339803ecb5cbb9 with merge 0b7d4cc6ff80b943953750e3bea782267bd22ff6...

bors avatar Mar 04 '24 08:03 bors

:sunny: Test successful - checks-actions Approved by: Veykril Pushing 0b7d4cc6ff80b943953750e3bea782267bd22ff6 to master...

bors avatar Mar 04 '24 09:03 bors