-
Notifications
You must be signed in to change notification settings - Fork 628
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
TOML: new PEG based parser #3509
Conversation
f8298d7
to
479fcda
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3509 +/- ##
==========================================
+ Coverage 85.45% 85.49% +0.03%
==========================================
Files 235 236 +1
Lines 56848 56970 +122
==========================================
+ Hits 48582 48705 +123
+ Misses 8266 8265 -1 ☔ View full report in Codecov by Sentry. |
d3bd6b0
to
da8e9f1
Compare
It seems that a bug in pegof. |
First problem I found is that pegof doesn't currently support unicode escapes in strings, e.g. |
This PEG parser is based on https://raw.githubusercontent.com/toml-lang/toml/1.0.0/toml.abnf
@dolik-rce Thank you. |
The peg file is imported from