Skip to content

Conversation

LucianoPAlmeida
Copy link
Contributor

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

Resolves SR-13359.

@LucianoPAlmeida LucianoPAlmeida force-pushed the SR-13359-subscript-tuple branch from 99a011e to 0d5bfcb Compare August 9, 2020 00:41
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?

@LucianoPAlmeida LucianoPAlmeida force-pushed the SR-13359-subscript-tuple branch from 681f8b1 to e8f1784 Compare August 10, 2020 02:20
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
Contributor Author

@swift-ci Please smoke test

@LucianoPAlmeida
Copy link
Contributor Author

@swift-ci Please smoke test Windows Platform

Copy link
Contributor

@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.

@LucianoPAlmeida
Copy link
Contributor Author

@swift-ci Please smoke test

@LucianoPAlmeida
Copy link
Contributor Author

@swift-ci Please smoke test Windows Platform

@LucianoPAlmeida LucianoPAlmeida force-pushed the SR-13359-subscript-tuple branch from 1f45291 to d62e9dd Compare August 11, 2020 00:06
@LucianoPAlmeida
Copy link
Contributor Author

@swift-ci Please smoke test

@LucianoPAlmeida
Copy link
Contributor Author

@swift-ci Please smoke test Windows Platform

@LucianoPAlmeida LucianoPAlmeida merged commit c0055e7 into swiftlang:master Aug 11, 2020
@LucianoPAlmeida LucianoPAlmeida deleted the SR-13359-subscript-tuple branch August 11, 2020 03:05
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.

4 participants