Skip to content

[Parse] Remove special handling for .<keyword><code-complete> #18869

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
Aug 22, 2018

Conversation

rintaro
Copy link
Member

@rintaro rintaro commented Aug 21, 2018

At the time this logic was introduced in 8f83ca6, <expr>.<keyword> wasn't allowed. Now that SE-0071 has been implemented, this logic doesn't provide any positive effects.

@rintaro
Copy link
Member Author

rintaro commented Aug 21, 2018

@swift-ci Please smoke test

Copy link
Contributor

@jrose-apple jrose-apple left a comment

Choose a reason for hiding this comment

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

I assume we have tests for this, so LGTM!

@rintaro rintaro force-pushed the parse-dot-kw-complete branch from a6bb4eb to 10e6ed7 Compare August 22, 2018 03:08
@rintaro
Copy link
Member Author

rintaro commented Aug 22, 2018

Well... Added test cases 💦
@swift-ci Please smoke test

At the time this logic was introduced in 8f83ca6, `<expr>.<keyword>` wasn't
allowed. Now that SE-0071 has been implemented, this logic doesn't provide any
positive effects.
@rintaro rintaro force-pushed the parse-dot-kw-complete branch from 10e6ed7 to fdcdeeb Compare August 22, 2018 03:11
@rintaro
Copy link
Member Author

rintaro commented Aug 22, 2018

@swift-ci Please smoke test

@rintaro rintaro merged commit 1155ade into swiftlang:master Aug 22, 2018
@rintaro rintaro deleted the parse-dot-kw-complete branch August 22, 2018 04:37
Copy link
Member

@DougGregor DougGregor left a comment

Choose a reason for hiding this comment

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

LGTM

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