Skip to content

[NameLookup] Collect implicit opaque GenericParams #64037

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

Conversation

angela-laar
Copy link
Contributor

@angela-laar angela-laar commented Mar 2, 2023

Plain protocols with generic type parameters should be collected as opaque types i.e. Collection<T>

Fixes: rdar://101405496

@angela-laar angela-laar requested a review from xedin as a code owner March 2, 2023 19:04
@angela-laar angela-laar force-pushed the fix-generic-type-paramters-implicit-some branch from efcd422 to 41392db Compare March 3, 2023 00:03
@angela-laar
Copy link
Contributor Author

@swift-ci please smoke test

@xedin xedin removed their request for review March 3, 2023 23:17
Plain protocols with generic type paramters should
be collected as opaque types.
@angela-laar angela-laar force-pushed the fix-generic-type-paramters-implicit-some branch from 41392db to ac00a8c Compare March 15, 2023 23:02
@angela-laar
Copy link
Contributor Author

@swift-ci please smoke test

@angela-laar angela-laar force-pushed the fix-generic-type-paramters-implicit-some branch from 6f230b0 to 68e1de4 Compare March 16, 2023 01:06
@angela-laar angela-laar force-pushed the fix-generic-type-paramters-implicit-some branch from 68e1de4 to 3a20ff0 Compare March 16, 2023 01:08
@angela-laar
Copy link
Contributor Author

@swift-ci please smoke test

@angela-laar angela-laar force-pushed the fix-generic-type-paramters-implicit-some branch from 71c42a6 to c3eb22a Compare March 21, 2023 21:46
@angela-laar
Copy link
Contributor Author

@swift-ci please smoke test

@angela-laar angela-laar merged commit 175e712 into swiftlang:main Mar 27, 2023
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