ripcordsoftware/libscriptobject

A C++ 11 library implementing scriptable/serializable objects

libscriptobject's Issues

Refactor get* code since there is a degree of duplication

The JSON factories don't convert to Int32 for numbers

If a number could fit into the int32 number space then the parser factory should attempt the downcast

Objects to support child objects

Replace shared_ptr with a lighter impl

std::shared_ptr<> instances require 16 bytes of memory. Switching to boost::intrusive_ptr would reduce this to 8 bytes. Where there are a large number of small objects/arrays the overhead of shared_ptr is high.

Rather than using boost implement as a new counted reference type.

Objects to support child arrays

Object creation to support key caching

Using shared_ptr<> outside new ScriptObject/ScriptArray results in a double allocation

ScriptObjectPtr and ScriptArrayPtr should be created by make_shared<> to avoid seperate object and control block allocations.

Tests: include fuzz tests for object keys

Need a lambda friendly unordered_set like find()

The std::unordered_set does not support a lambda implementation of find(). This means the template type must be created to get a key comparison. In the key cache case the type is a shared_ptr which is a relatively expensive object. It is much preferred to compare the field count and the hash without the expense of creating a smart pointer instance.

Add support for set*()

Support for setString() was added in 6f98352. It would be useful to set the value of other fields in the object. Also consider set for array.

Complete the wiki pages

Add doxygen comments to the code

Add dynamic information into the exceptions

Support single array types

If any array contains a single type then type and offset information may be omitted which is a useful optimization for large arrays.

Tests: include fuzz tests for objects

Return total object allocated size

Object/array to return the sum total of self and children to caller. Both object and array already expose self 'size' but there is no child walk function built in.