Skip to content

Fixup warnings in tests #7963

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 13, 2024
Merged

Conversation

plemarquand
Copy link
Contributor

Fixes up a few different types of warnings in the tests:

Fixes up a few different types of warnings in the tests:
- Unused variables are replaced with `_`
- Continuation handlers are not passed through directly (follows on
  from swiftlang#7961)
- Removes unnecessary `await`s from functions that are now synchronous
@plemarquand
Copy link
Contributor Author

There are just a few warnings left in the Tests, which look like a compiler issue to me?

For instance the line:

extension Term: @retroactive ExpressibleByStringLiteral {}

warns with:
'retroactive' attribute does not apply; 'ExpressibleByStringLiteral' is declared in this module; this is an error in the Swift 6 language mode

ExpressibleByStringLiteral is a core Swift type and so not defined in this module. Anyone know why this warning appears?

@MaxDesiatov
Copy link
Contributor

ExpressibleByStringLiteral is a core Swift type and so not defined in this module. Anyone know why this warning appears?

looks like a compiler bug to me

@MaxDesiatov
Copy link
Contributor

@swift-ci test

@plemarquand
Copy link
Contributor Author

@swift-ci test Windows platform

@MaxDesiatov MaxDesiatov merged commit e3721b3 into swiftlang:main Sep 13, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants