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

Support for HJSON #1377

Open
Yzrsah opened this issue Oct 12, 2018 · 1 comment
Open

Support for HJSON #1377

Yzrsah opened this issue Oct 12, 2018 · 1 comment

Comments

@Yzrsah
Copy link

Yzrsah commented Oct 12, 2018

Hello,

My app uses many data configuration files.

The files are in the HJSON format. Due to performance concerns, we attempted to switch from HJSON to JSON utilizing rapidjson.

The users of the app were very unhappy, as adding the operators and extra symbols all around significantly complicated the files. I got countless requests to put the old behavior back with HJSON. It basically ruined the elegance and simplicity of what the users were used to being able to write.

The JSON format is good for serialization, but it is not the best for configuration and data authoring due to the requirement for excessive operator usage.

Due to this, I basically have no choice. Either I use the slow implementation of 'hjson-cpp' or the rapidjson needs support for some relaxed format.

Perhaps some sort of modular profile system capable of supporting multiple relaxed syntaxes could work. There's a few different relaxed JSON syntaxes but they are pretty similar, all with the same important additions like the ability to use comments.

@miloyip
Copy link
Collaborator

miloyip commented Oct 16, 2018

RapidJSON does support some relaxed syntax, See #36.
But it may not support all the syntax required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants