Skip to content

[AST] Resolved name quotation inconsistency #1015

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

Merged
merged 1 commit into from
Jan 20, 2016
Merged

Conversation

mteper
Copy link
Contributor

@mteper mteper commented Jan 19, 2016

Prior to this change, the AST would look something like this, note id='incoming missing the closing quote.

(type_named id='incoming
    (type_ident
        (component id='UIImage' bind=type)))

In reviewing conventions used throughout this class, it appears that this identifier should not need to be quoted at all, so I removed the leading quote. If I am wrong in my understanding, then perhaps the trailing quote should be introduced.

Prior to this change, the AST would look something like this, note `id='incoming` missing the closing quote.

```swift
(type_named id='incoming
    (type_ident
        (component id='UIImage' bind=type)))
```

In reviewing conventions used throughout this class, it appears that this identifier should not need to be quoted at all, so I removed the leading quote. If I am wrong in my understanding, then perhaps the trailing quote should be introduced.
@mteper mteper changed the title Resolved name quotation inconsistency [AST] Resolved name quotation inconsistency Jan 19, 2016
lattner added a commit that referenced this pull request Jan 20, 2016
[AST] Resolved name quotation inconsistency
@lattner lattner merged commit dc333cf into swiftlang:master Jan 20, 2016
@mteper mteper deleted the patch-1 branch January 20, 2016 19:42
MaxDesiatov added a commit to MaxDesiatov/swift that referenced this pull request May 20, 2020
Update wasm branch scheme to reflect upstream master
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.

2 participants