-
-
Notifications
You must be signed in to change notification settings - Fork 391
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
Default values & serialization #196
Comments
I have just added API to set boolean value to control serialize default value or not.
Before the RapidJson transition, I'm considering to let tinygltf co-exist rapidjson and json.hpp in |
Continued discussion from #193 .
I see. I don't think I'll get around to change this any time soon, but let's discuss it anyway (for others).
The mid-term solution is logical. It is not a lot of extra work to begin with this; simply a boolean or define to set? I remember you weren't fond of adding more defines in the past, do you have a preferred approach to that?
The long-term solution is nice, but that seems quite far away indeed. Let's get RapidJSON to work first. 👍
The text was updated successfully, but these errors were encountered: