think about fjson_object_to_json_string_fn type

[rgerhards]

Do we really need level and flags parameters in all functions? Should we replace the function pointers with a switch() statement based on the object type?

[davidelang]

It's a good idea to have flags as future-proofing (there's a good lwn.net article on the topic)

can you point to an example of what you are looking at?

[rgerhards]

Here we have the functions: https://github.com/rsyslog/libfastjson/blob/master/json_object.c#L50

While it is good to have future-proof, in our regard that means we need to write 8 bytes on the stack for each recursive function call (this is one of the hot spots, but less then malloc calls). I am undecided, maybe profiler should suggest the way forward. Alos, this would require some feature-loss. It's something to later look at.