Skip to content
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

[SR-13359] [Sema] Tailored diagnostics for missing subscript members on tuple type base #33380

Merged

Conversation

LucianoPAlmeida
Copy link
Collaborator

This adds a tailored diagnostic for missing subscript members on tuple type base.

Resolves SR-13359.

Copy link
Contributor

@varungandhi-apple varungandhi-apple left a comment

Choose a reason for hiding this comment

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

Slightly concerned about the diagnostic wording here. Also, should we have additional handling + tests here for labelled tuples?

Copy link
Contributor

@varungandhi-apple varungandhi-apple left a comment

Choose a reason for hiding this comment

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

LGTM.

@varungandhi-apple
Copy link
Contributor

@swift-ci please smoke test

@LucianoPAlmeida
Copy link
Collaborator Author

@swift-ci Please smoke test

@LucianoPAlmeida
Copy link
Collaborator Author

@swift-ci Please smoke test Windows Platform

Copy link
Member

@xedin xedin left a comment

Choose a reason for hiding this comment

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

Looks great, thank you, @LucianoPAlmeida! I have left a couple of minor notes inline.

lib/Sema/CSDiagnostics.cpp Outdated Show resolved Hide resolved
lib/Sema/CSDiagnostics.cpp Outdated Show resolved Hide resolved
localization/diagnostics/en.yaml Outdated Show resolved Hide resolved
@LucianoPAlmeida
Copy link
Collaborator Author

@swift-ci Please smoke test

@LucianoPAlmeida
Copy link
Collaborator Author

@swift-ci Please smoke test Windows Platform

lib/Sema/CSDiagnostics.cpp Outdated Show resolved Hide resolved
@LucianoPAlmeida
Copy link
Collaborator Author

@swift-ci Please smoke test

@LucianoPAlmeida
Copy link
Collaborator Author

@swift-ci Please smoke test Windows Platform

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.

None yet

4 participants