cesium-native icon indicating copy to clipboard operation
cesium-native copied to clipboard

Improvements to the JsonValue type

Open kring opened this issue 4 years ago • 0 comments

The JsonValue type was thrown together rather quickly to support glTF extras and not really given a lot of careful consideration. We should make sure it's an interface we're happy to commit to long-term.

We should consider:

  • Is it ever ok to throw exceptions? When?
  • Is it important to have an integer type distinct from a double type? What about signed versus unsigned? What about big integers?
  • Is our interface clear, easily discoverable, and hard to misuse? Look at https://github.com/nlohmann/json for interface inspiration, though we probably don't need or want to support that entire feature set.
  • Is the performance and memory usage acceptable?

kring avatar Apr 27 '21 07:04 kring