gbowne1 / gbowne1/jsonify

[TODO] Enhance memory safety

Open
#14 0 comments 0 reactions 0 assignees View on GitHub
bug documentation enhancement good first issue help wanted question
Dominant language
C++
Stars
4
Forks
6
PR merge metrics
No merged PRs in 30d

Description

We are using std::shared_ptr correctly for the tree structure. However, the JsonValue class stores all possible types (bool, number, string, array, object) as separate members.

In a production-grade parser, these are usually stored in a std::variant or a union to save memory, as a single JSON value can only be one type at a time.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.