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.

Sign up for GitHub

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

Jump to bottom

Migrate to RapidJSON #18

Closed
apjanke opened this issue Mar 9, 2020 · 1 comment
Closed

Migrate to RapidJSON #18

apjanke opened this issue Mar 9, 2020 · 1 comment
Labels
enhancement New feature or request
Projects
jsonstuff
Milestone
1.0

Comments

@apjanke
Copy link
Owner

apjanke commented Mar 9, 2020

We're currently using JsonCPP as the underlying JSON parsing/manipulation library. RapidJSON looks like it will be a better library: faster, higher quality, and better documented (per #4).

Migrate JsonStuff to use JsonCPP.
@apjanke apjanke added the enhancement New feature or request label Mar 9, 2020
@apjanke apjanke added this to Needs triage in jsonstuff via automation Mar 9, 2020
@apjanke apjanke moved this from Needs triage to Low priority in jsonstuff Mar 9, 2020
@apjanke apjanke added this to the 1.0 milestone Mar 9, 2020
This was referenced Mar 9, 2020
Closed
Open
Open
@apjanke
Copy link
Owner Author

apjanke commented Mar 9, 2020

Done in 83cbf12.

@apjanke apjanke closed this as completed Mar 9, 2020
jsonstuff automation moved this from Low priority to Closed Mar 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
jsonstuff
  
Closed
Milestone
1.0
Development

No branches or pull requests
1 participant
@apjanke