Skip to content

Conversation

dcci
Copy link
Member

@dcci dcci commented Dec 7, 2018

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@dcci dcci requested a review from slavapestov December 7, 2018 01:32
@dcci
Copy link
Member Author

dcci commented Dec 7, 2018

@swift-ci test and merge

@dcci
Copy link
Member Author

dcci commented Dec 7, 2018

@slavapestov oops, I just read your comment after submitting the PR.

@@ -3,7 +3,6 @@ $SytD ---> ()
$Ss5Int32VD ---> Int32
$S4blah4mainyyF8PatatinoL_VMa ---> Can't resolve type of $S4blah4mainyyF8PatatinoL_VMa
$Ss10CollectionP7Element ---> Can't resolve type of $Ss10CollectionP7Element
$Ss15ContiguousArrayV9formIndex5afterySiz_tFSS_Tg5 ---> (inout Int) -> ()
Copy link
Contributor

Choose a reason for hiding this comment

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

No, this needs to continue to work since its still valid to have values whose types are function types with inout parameters.

@swift-ci swift-ci merged commit f98b617 into swiftlang:master Dec 7, 2018
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