Skip to content

[5.7][CodeCompletion] Teach TypeCheckASTNodeAtLocRequest to check initializers #42185

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

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Apr 5, 2022

Cherry-pick #42163 to release/5.7.


This simplifies the logic in ide::typeCheckContextAt and fixes an issue that prevent code completion from working inside variables that are initialized by calling a closure.

Fixes rdar://90455154 [SR-16012]
Fixes rdar://85600167 [SR-15495]

@ahoppen ahoppen requested a review from xedin April 5, 2022 15:54
@ahoppen ahoppen changed the title [CodeCompletion] Teach TypeCheckASTNodeAtLocRequest to check initializers [5.7][CodeCompletion] Teach TypeCheckASTNodeAtLocRequest to check initializers Apr 5, 2022
@ahoppen
Copy link
Member Author

ahoppen commented Apr 5, 2022

@swift-ci Please test

@xedin
Copy link
Contributor

xedin commented Apr 5, 2022

@swift-ci please test macOS platform

@xedin
Copy link
Contributor

xedin commented Apr 5, 2022

@swift-ci please test Windows platform

@xedin
Copy link
Contributor

xedin commented Apr 5, 2022

@swift-ci please test Linux platform

@ahoppen ahoppen added the r5.7 label Apr 7, 2022
@ahoppen
Copy link
Member Author

ahoppen commented Apr 7, 2022

@swift-ci Please test macOS

@ahoppen ahoppen requested a review from airspeedswift April 8, 2022 06:57
…zers

This simplifies the logic in `ide::typeCheckContextAt` and fixes an issue that prevent code completion from working inside variables that are initialized by calling a closure.

Fixes rdar://90455154 [SR-16012]
Fixes rdar://85600167 [SR-15495]
@ahoppen ahoppen force-pushed the pr/fix-no-completion-in-closure-in-init branch from cbbca38 to 0b3f2cc Compare April 13, 2022 06:18
@ahoppen ahoppen requested a review from a team as a code owner April 13, 2022 06:18
@ahoppen
Copy link
Member Author

ahoppen commented Apr 13, 2022

@swift-ci Please test

@ahoppen ahoppen merged commit 77f676b into swiftlang:release/5.7 Apr 20, 2022
@ahoppen ahoppen deleted the pr/fix-no-completion-in-closure-in-init branch April 20, 2022 19:06
@AnthonyLatsis AnthonyLatsis added 🍒 release cherry pick Flag: Release branch cherry picks swift 5.7 labels Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks swift 5.7
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants