Skip to content

Conversation

@xedin
Copy link
Contributor

@xedin xedin commented Nov 29, 2025

…apper in swift interfaces

The type of an outermost property wrapper should be printed together with (inferred) generic arguments because otherwise if a any wrapper in the chain has generic parameters that are not involved in wrappedValue: argument it won't be impossible to infer them while type-checking the interface file where it appears.

Resolves: rdar://122963120

…apper in swift interfaces

The type of an outermost property wrapper should be printed together
with (inferred) generic arguments because otherwise if a any wrapper
in the chain has generic parameters that are not involved in
`wrappedValue:` argument it won't be impossible to infer them while
type-checking the interface file where it appears.

Resolves: rdar://122963120
@xedin
Copy link
Contributor Author

xedin commented Nov 29, 2025

@swift-ci please test

@xedin
Copy link
Contributor Author

xedin commented Nov 29, 2025

@swift-ci please test source compatibility

@xedin
Copy link
Contributor Author

xedin commented Nov 29, 2025

@swift-ci please test Windows platform

1 similar comment
@xedin
Copy link
Contributor Author

xedin commented Nov 30, 2025

@swift-ci please test Windows platform

@nkcsgexi
Copy link
Contributor

nkcsgexi commented Dec 1, 2025

Thank you for fixing this, Pavel!

Copy link
Contributor

@hamishknight hamishknight left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think ideally we'd fix this by setting the resolved type for the TypeExpr in CSApply, but happy to land this

@xedin
Copy link
Contributor Author

xedin commented Dec 1, 2025

Yeah, I can look into that separately, decided to make this targeted.

@xedin
Copy link
Contributor Author

xedin commented Dec 2, 2025

@swift-ci please test Windows platform

@xedin xedin merged commit b6291e6 into swiftlang:main Dec 2, 2025
7 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.

3 participants