Skip to content

Account for Deserialized VarDecls in Setter Mismatch Diagnostics #31961

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
May 22, 2020

Conversation

CodaFi
Copy link
Contributor

@CodaFi CodaFi commented May 22, 2020

If the setter conflict occurs in a deserialized declaration, the parent
pattern binding can be NULL. Guard the fixit on the existence of the
pattern binding so

  1. we don't crash
  2. we don't try to emit a fixit in otherwise extremely broken code

rdar://56558082

If the setter conflict occurs in a deserialized declaration, the parent
pattern binding can be NULL. Guard the fixit on the existence of the
pattern binding so

1) we don't crash
2) we don't try to emit a fixit in otherwise extremely broken code

rdar://56558082
@CodaFi CodaFi requested a review from theblixguy May 22, 2020 02:22
@CodaFi
Copy link
Contributor Author

CodaFi commented May 22, 2020

@swift-ci smoke test and merge

@swift-ci swift-ci merged commit 1f85c04 into swiftlang:master May 22, 2020
@CodaFi CodaFi deleted the setternalia branch May 26, 2020 18:12
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.

2 participants