Skip to content

Conversation

@gottesmm
Copy link
Contributor

I have noticed over time when working on the command line, we often times highlight too large of an expression due to the locations provided to us by earlier parts of the compiler. This isn't technically necessary and the following doesn't look nice... so remove it.

test5.swift:171:16: error: sending 'x' risks causing data races [#SendingRisksDataRace]
169 |   let _ = {
170 |     nonescapingAsyncUse { @MainActor in
171 | _� _� _� _� _�u_�s_�e_�V_�a_�l_�u_�e_�(_�x_�)
    |                |- error: sending 'x' risks causing data races [#SendingRisksDataRace]
    |                `- note: task-isolated 'x' is captured by a main actor-isolated closure. main actor-isolated uses in closure may race against later nonisolated uses
172 |     }
173 |   }

I have noticed over time when working on the command line, we often times
highlight too large of an expression due to the locations provided to us by
earlier parts of the compiler. This isn't technically necessary and the
following doesn't look nice... so remove it.

```
test5.swift:171:16: error: sending 'x' risks causing data races [#SendingRisksDataRace]
169 |   let _ = {
170 |     nonescapingAsyncUse { @mainactor in
171 | _� _� _� _� _�u_�s_�e_�V_�a_�l_�u_�e_�(_�x_�)
    |                |- error: sending 'x' risks causing data races [#SendingRisksDataRace]
    |                `- note: task-isolated 'x' is captured by a main actor-isolated closure. main actor-isolated uses in closure may race against later nonisolated uses
172 |     }
173 |   }
```
@gottesmm gottesmm requested a review from eeckstein as a code owner December 12, 2025 01:29
@gottesmm gottesmm enabled auto-merge December 12, 2025 01:29
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test

@gottesmm gottesmm merged commit 79eee82 into swiftlang:main Dec 12, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant