Skip to content

Conversation

nathawes
Copy link
Contributor

@nathawes nathawes commented Sep 17, 2019

This simplifies the code and prevents an assertion failure this code was hitting computing the type relations between the result types and what it determined as the expected type.

Resolves rdar://problem/53958454

@nathawes nathawes requested a review from rintaro September 17, 2019 21:41
@nathawes
Copy link
Contributor Author

@swift-ci please test

@nathawes nathawes force-pushed the r53958454-getFixedTypeRecursive-code-completion-crash branch from 4e26659 to a9c07cd Compare September 18, 2019 01:18
@nathawes
Copy link
Contributor Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 4e266598b906bd7205ea00c85bc333c6f8d7b6f9

@nathawes nathawes changed the title [code-completion] Don't set an expected type for the RHS of an assignment if the LHS is not assignable [code-completion] Remove special handling for completion on the RHS of an assignment Sep 18, 2019
@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 4e266598b906bd7205ea00c85bc333c6f8d7b6f9

…f an assignment

This simplifies the code and prevents an assertion failure this code was
hitting computing the type relations between the result types and what it
determined as the expected type.

Resolves rdar://problem/53958454
@nathawes nathawes force-pushed the r53958454-getFixedTypeRecursive-code-completion-crash branch from a9c07cd to 54defbc Compare September 18, 2019 01:59
@nathawes
Copy link
Contributor Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - a9c07cd6c8018439d9d4c76d8c068efa3ac7e312

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - a9c07cd6c8018439d9d4c76d8c068efa3ac7e312

Copy link
Member

@rintaro rintaro left a comment

Choose a reason for hiding this comment

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

Nice simplification!

@nathawes
Copy link
Contributor Author

🙇‍♂️ Thanks for suggesting it @rintaro!

@nathawes nathawes merged commit 4dac0d1 into swiftlang:master Sep 18, 2019
@nathawes nathawes deleted the r53958454-getFixedTypeRecursive-code-completion-crash branch September 18, 2019 17:49
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