JSON not formatted anymore

[Robadob]

Discussed in #1204

^{Originally posted by bytetec May 13, 2024}
Hi everyone
Since the update from FLAME GPU 2.0.0-rc to FLAME GPU 2.0.0-rc.1 the export of the logfile in JSON produces only a one liner. Does anybody else has this issue?
Before:

After:

---

It appears the methods of RapidJSON's Writer are not virtual, hence when a pointer to the PrettyWriter subclass is cast to Writer, method calls are not forwarded to the sub class.

FLAMEGPU2/src/flamegpu/io/JSONStateWriter.cu

Line 33 in b5173e7

writer = std::move(t_writer);

This bug was introduced when the StateWriter interface was refactored to better enable partial state writes.

Solution would probably be to template JSONStateWriter so that the two different printers have distinct objects.