Skip to content

Conversation

LucianoPAlmeida
Copy link
Contributor

This simply warns about dictionary literal with duplicated keys only when they are inferred as Dictionary where we know duplicated keys are not stored, but we cannot diagnose the same for other custom ExpressibleByDictionaryLiteral type because rules are unknown.

Resolves #43258.

@LucianoPAlmeida LucianoPAlmeida requested a review from xedin August 10, 2022 00:58
@LucianoPAlmeida
Copy link
Contributor Author

@swift-ci Please test

@LucianoPAlmeida
Copy link
Contributor Author

@swift-ci Please test macOS 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.

Overall looks great, thank you! I left a couple comments inline.

@LucianoPAlmeida LucianoPAlmeida requested a review from xedin August 13, 2022 04:29
@LucianoPAlmeida LucianoPAlmeida force-pushed the dictionary-literal branch 3 times, most recently from 054baa4 to 2d1bcd2 Compare August 14, 2022 00:16
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! I have left a couple of comments inline, no need for re-review.

@LucianoPAlmeida LucianoPAlmeida force-pushed the dictionary-literal branch 2 times, most recently from c9c7324 to 30049fc Compare August 16, 2022 01:31
@LucianoPAlmeida
Copy link
Contributor Author

@swift-ci Please test

@LucianoPAlmeida LucianoPAlmeida merged commit 1b83307 into swiftlang:main Aug 16, 2022
@LucianoPAlmeida LucianoPAlmeida deleted the dictionary-literal branch August 18, 2022 12:16
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.

[SR-641] [QOI] Missing diagnostic for static dictionary literal w/ duplicate keys
2 participants