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

Make the parser compliant with JSON5 specs #40

Open
1 of 2 tasks
aqrln opened this issue May 10, 2017 · 1 comment
Open
1 of 2 tasks

Make the parser compliant with JSON5 specs #40

aqrln opened this issue May 10, 2017 · 1 comment
Assignees

Comments

aqrln referenced this issue in metarhia/jstp May 12, 2017
This commit adds and adapts the official JSON5 parser tests.
`test/todo/json5.js` should be moved to `test/node` as soon as we fix
all the failures.

PR-URL: #158
Refs: https://github.com/metarhia/JSTP/issues/152
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Mykola Bilochub <nbelochub@gmail.com>
nechaido referenced this issue in metarhia/jstp May 18, 2017
This commit adds and adapts the official JSON5 parser tests.
`test/todo/json5.js` should be moved to `test/node` as soon as we fix
all the failures.

PR-URL: #158
Refs: https://github.com/metarhia/JSTP/issues/152
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Mykola Bilochub <nbelochub@gmail.com>
aqrln referenced this issue in metarhia/jstp Jun 15, 2017
Refs: #175
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: #221
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
@belochub
Copy link
Member

@metarhia/jstp-core, I've updated the description to represent current progress on this issue.

aqrln referenced this issue in metarhia/jstp Jun 20, 2017
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: #223
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
aqrln referenced this issue in metarhia/jstp Jun 21, 2017
* Add tool for getting and parsing needed categories from Unicode
  Character Database and generating C++ header file with code points
  arrays.

* Add UTF-8 decoding function.

* Implement Unicode keys parsing and add two possible options to use
  when checking whether the code point is an identifier.

Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: #218
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
aqrln referenced this issue in metarhia/jstp Jun 26, 2017
* Implement surrogate Unicode pairs parsing.
* Parse Unicode escape sequences in objects.
* Rework ReadHexNumber function.
* Improve key parsing performance by avoiding copying when possible.

PR-URL: #219
Refs: https://github.com/metarhia/jstp/issues/152
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue in metarhia/jstp Jun 28, 2017
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: #239
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
belochub referenced this issue in metarhia/jstp Jul 7, 2017
Also, deprecate legacy octal escape sequences in strings.

Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: #247
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
@belochub belochub changed the title Make the parser and serializer compliant with JSON5 specs Make the parser compliant with JSON5 specs Jul 10, 2017
belochub referenced this issue in metarhia/jstp Jan 22, 2018
This commit adds and adapts the official JSON5 parser tests.
`test/todo/json5.js` should be moved to `test/node` as soon as we fix
all the failures.

PR-URL: #158
Refs: https://github.com/metarhia/JSTP/issues/152
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Mykola Bilochub <nbelochub@gmail.com>
belochub referenced this issue in metarhia/jstp Jan 22, 2018
Refs: #175
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: #221
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue in metarhia/jstp Jan 22, 2018
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: #223
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
belochub referenced this issue in metarhia/jstp Jan 22, 2018
* Add tool for getting and parsing needed categories from Unicode
  Character Database and generating C++ header file with code points
  arrays.

* Add UTF-8 decoding function.

* Implement Unicode keys parsing and add two possible options to use
  when checking whether the code point is an identifier.

Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: #218
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue in metarhia/jstp Jan 22, 2018
* Implement surrogate Unicode pairs parsing.
* Parse Unicode escape sequences in objects.
* Rework ReadHexNumber function.
* Improve key parsing performance by avoiding copying when possible.

PR-URL: #219
Refs: https://github.com/metarhia/jstp/issues/152
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue in metarhia/jstp Jan 22, 2018
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: #239
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
belochub referenced this issue in metarhia/jstp Jan 22, 2018
Also, deprecate legacy octal escape sequences in strings.

Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: #247
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue in metarhia/jstp Jan 22, 2018
This commit adds and adapts the official JSON5 parser tests.
`test/todo/json5.js` should be moved to `test/node` as soon as we fix
all the failures.

PR-URL: #158
Refs: https://github.com/metarhia/JSTP/issues/152
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Mykola Bilochub <nbelochub@gmail.com>
belochub referenced this issue in metarhia/jstp Jan 22, 2018
Refs: #175
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: #221
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue in metarhia/jstp Jan 22, 2018
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: #223
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
belochub referenced this issue in metarhia/jstp Jan 22, 2018
* Add tool for getting and parsing needed categories from Unicode
  Character Database and generating C++ header file with code points
  arrays.

* Add UTF-8 decoding function.

* Implement Unicode keys parsing and add two possible options to use
  when checking whether the code point is an identifier.

Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: #218
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue in metarhia/jstp Jan 22, 2018
* Implement surrogate Unicode pairs parsing.
* Parse Unicode escape sequences in objects.
* Rework ReadHexNumber function.
* Improve key parsing performance by avoiding copying when possible.

PR-URL: #219
Refs: https://github.com/metarhia/jstp/issues/152
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue in metarhia/jstp Jan 22, 2018
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: #239
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
belochub referenced this issue in metarhia/jstp Jan 22, 2018
Also, deprecate legacy octal escape sequences in strings.

Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: #247
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue Jul 19, 2018
This commit adds and adapts the official JSON5 parser tests.
`test/todo/json5.js` should be moved to `test/node` as soon as we fix
all the failures.

PR-URL: metarhia/jstp#158
Refs: https://github.com/metarhia/JSTP/issues/152
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Mykola Bilochub <nbelochub@gmail.com>
belochub referenced this issue Jul 19, 2018
Refs: metarhia/jstp#175
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: metarhia/jstp#221
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue Jul 19, 2018
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: metarhia/jstp#223
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
belochub referenced this issue Jul 19, 2018
* Add tool for getting and parsing needed categories from Unicode
  Character Database and generating C++ header file with code points
  arrays.

* Add UTF-8 decoding function.

* Implement Unicode keys parsing and add two possible options to use
  when checking whether the code point is an identifier.

Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: metarhia/jstp#218
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue Jul 19, 2018
* Implement surrogate Unicode pairs parsing.
* Parse Unicode escape sequences in objects.
* Rework ReadHexNumber function.
* Improve key parsing performance by avoiding copying when possible.

PR-URL: metarhia/jstp#219
Refs: https://github.com/metarhia/jstp/issues/152
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue Jul 19, 2018
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: metarhia/jstp#239
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
belochub referenced this issue Jul 19, 2018
Also, deprecate legacy octal escape sequences in strings.

Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: metarhia/jstp#247
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue Jul 19, 2018
This commit adds and adapts the official JSON5 parser tests.
`test/todo/json5.js` should be moved to `test/node` as soon as we fix
all the failures.

PR-URL: metarhia/jstp#158
Refs: https://github.com/metarhia/JSTP/issues/152
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Mykola Bilochub <nbelochub@gmail.com>
belochub referenced this issue Jul 19, 2018
Refs: metarhia/jstp#175
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: metarhia/jstp#221
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue Jul 19, 2018
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: metarhia/jstp#223
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
belochub referenced this issue Jul 19, 2018
* Add tool for getting and parsing needed categories from Unicode
  Character Database and generating C++ header file with code points
  arrays.

* Add UTF-8 decoding function.

* Implement Unicode keys parsing and add two possible options to use
  when checking whether the code point is an identifier.

Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: metarhia/jstp#218
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue Jul 19, 2018
* Implement surrogate Unicode pairs parsing.
* Parse Unicode escape sequences in objects.
* Rework ReadHexNumber function.
* Improve key parsing performance by avoiding copying when possible.

PR-URL: metarhia/jstp#219
Refs: https://github.com/metarhia/jstp/issues/152
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue Jul 19, 2018
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: metarhia/jstp#239
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
belochub referenced this issue Jul 19, 2018
Also, deprecate legacy octal escape sequences in strings.

Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: metarhia/jstp#247
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue Jul 21, 2018
This commit adds and adapts the official JSON5 parser tests.
`test/todo/json5.js` should be moved to `test/node` as soon as we fix
all the failures.

PR-URL: metarhia/jstp#158
Refs: https://github.com/metarhia/JSTP/issues/152
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Mykola Bilochub <nbelochub@gmail.com>
belochub referenced this issue Jul 21, 2018
Refs: metarhia/jstp#175
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: metarhia/jstp#221
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue Jul 21, 2018
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: metarhia/jstp#223
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
belochub referenced this issue Jul 21, 2018
* Add tool for getting and parsing needed categories from Unicode
  Character Database and generating C++ header file with code points
  arrays.

* Add UTF-8 decoding function.

* Implement Unicode keys parsing and add two possible options to use
  when checking whether the code point is an identifier.

Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: metarhia/jstp#218
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue Jul 21, 2018
* Implement surrogate Unicode pairs parsing.
* Parse Unicode escape sequences in objects.
* Rework ReadHexNumber function.
* Improve key parsing performance by avoiding copying when possible.

PR-URL: metarhia/jstp#219
Refs: https://github.com/metarhia/jstp/issues/152
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
belochub referenced this issue Jul 21, 2018
Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: metarhia/jstp#239
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
belochub referenced this issue Jul 21, 2018
Also, deprecate legacy octal escape sequences in strings.

Refs: https://github.com/metarhia/jstp/issues/152
PR-URL: metarhia/jstp#247
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Dmytro Nechai <nechaido@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
@belochub belochub transferred this issue from metarhia/jstp Feb 12, 2019
@JuliaGerasymenko JuliaGerasymenko self-assigned this Feb 19, 2019
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

No branches or pull requests

3 participants