Skip to content
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

Allow ByteTree deserialisation functions to throw #2

Merged
merged 1 commit into from
Sep 11, 2018

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Aug 30, 2018

At the moment we fatalError if an error occurs while deserialising ByteTree (e.g. because of an unexpected syntax node kind or because incremental parsing failed. With this, we will throw and the client will be able to handle the error.

@rintaro
Copy link
Member

rintaro commented Sep 11, 2018

@nkcsgexi nkcsgexi merged commit 239c66a into swiftlang:master Sep 11, 2018
@ahoppen ahoppen deleted the throwing-decoding branch September 12, 2018 07:26
CippoX added a commit to CippoX/swift-syntax that referenced this pull request Mar 21, 2023
# This is the 1st commit message:

fixed testAvailabilityQuery34 and testAvailabilityQueryUnavailability28

# This is the commit message swiftlang#2:

Update Sources/SwiftParserDiagnostics/ParseDiagnosticsGenerator.swift

Co-authored-by: Kim de Vos <kimdevos12@hotmail.com>
# This is the commit message swiftlang#3:

added fixedSource in test case

# This is the commit message swiftlang#4:

minor changes

# This is the commit message swiftlang#5:

implemented recovery inside the parser

# This is the commit message swiftlang#6:

runned format.py

# This is the commit message swiftlang#7:

minor changes

# This is the commit message swiftlang#8:

minor changes
adevress pushed a commit to adevress/swift-syntax that referenced this pull request Jan 14, 2024
Remove extra space for closures with captures but no arguments.
kateinoigakukun pushed a commit to kateinoigakukun/swift-syntax that referenced this pull request Feb 27, 2024
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