diff options
| author | y21 <30553356+y21@users.noreply.github.com> | 2024-10-20 19:00:55 +0200 |
|---|---|---|
| committer | y21 <30553356+y21@users.noreply.github.com> | 2024-10-20 19:00:55 +0200 |
| commit | 65eb1ec0fbc72df2fa6122fa061cd73fc7140612 (patch) | |
| tree | 027f016dad5a87cf4731bebdb738a5cc3ab580ad /tests | |
| parent | 4de65a113f3f15cfc674c7b77cfa3afe1ddbe090 (diff) | |
| download | rust-65eb1ec0fbc72df2fa6122fa061cd73fc7140612.tar.gz rust-65eb1ec0fbc72df2fa6122fa061cd73fc7140612.zip | |
remove the semicolon for builtin macro call statements in `statement_outside_block`
The expansion of `asm!()` and `line!()` is not marked as from an expansion, in which case `SourceMap::stmt_span` returns the input span unchanged. So instead of using `stmt_span`, use `mac_call_stmt_semi_span` directly
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/ui/semicolon_outside_block.fixed | 8 | ||||
| -rw-r--r-- | tests/ui/semicolon_outside_block.rs | 8 | ||||
| -rw-r--r-- | tests/ui/semicolon_outside_block.stderr | 30 |
3 files changed, 45 insertions, 1 deletions
diff --git a/tests/ui/semicolon_outside_block.fixed b/tests/ui/semicolon_outside_block.fixed index 148e112e0bc..ac7e86631ca 100644 --- a/tests/ui/semicolon_outside_block.fixed +++ b/tests/ui/semicolon_outside_block.fixed @@ -80,5 +80,13 @@ fn main() { { unit_fn_block(); }; + unsafe { + std::arch::asm!("") + }; + + { + line!() + }; + unit_fn_block() } diff --git a/tests/ui/semicolon_outside_block.rs b/tests/ui/semicolon_outside_block.rs index c767201469a..68f25339e32 100644 --- a/tests/ui/semicolon_outside_block.rs +++ b/tests/ui/semicolon_outside_block.rs @@ -80,5 +80,13 @@ fn main() { { unit_fn_block(); }; + unsafe { + std::arch::asm!(""); + } + + { + line!(); + } + unit_fn_block() } diff --git a/tests/ui/semicolon_outside_block.stderr b/tests/ui/semicolon_outside_block.stderr index 68b44c8f980..ff8c00048f6 100644 --- a/tests/ui/semicolon_outside_block.stderr +++ b/tests/ui/semicolon_outside_block.stderr @@ -51,5 +51,33 @@ LL - { m!(()); } LL + { m!(()) }; | -error: aborting due to 4 previous errors +error: consider moving the `;` outside the block for consistent formatting + --> tests/ui/semicolon_outside_block.rs:83:5 + | +LL | / unsafe { +LL | | std::arch::asm!(""); +LL | | } + | |_____^ + | +help: put the `;` here + | +LL ~ std::arch::asm!("") +LL ~ }; + | + +error: consider moving the `;` outside the block for consistent formatting + --> tests/ui/semicolon_outside_block.rs:87:5 + | +LL | / { +LL | | line!(); +LL | | } + | |_____^ + | +help: put the `;` here + | +LL ~ line!() +LL ~ }; + | + +error: aborting due to 6 previous errors |
