aikawayataro

Alternative solution without source patching: use .js config with this code: ```js // BigInt JSON serialization. BigInt.prototype.toJSON = function() { return this.toString() + 'n'; } ``` Still monkey patch too...

`GenericDocument`'s destructor is not recursive it `delete`'s the allocator and that's it. The default allocator `MemoryPoolAllocator`'s destructor is not recursive too. But `GenericDocument` inherits `GenericValue` whose destructor is recursive. So...

Okay, I understand your point. The problem is that in order to implement a new flag for a value, the internal API have to be changed, including the `GenericValue` constructor,...

A simpler and faster code can also be achieved by using `GenericDocument::Populate`, but in case of parsing error you have to "close" all incomplete nodes, such as key-values, objects, and...

I think you should submit a pull request then. To avoid misunderstandings, I am not the maintainer of this repository. You need to define your destroyer in the `internal` namespace,...

It's possible to implement custom reader (points of interest are `ParseValue`, `ParseTrue` and `ParseFalse`) in C++ to implement this, but it seems you're using python. The most simple thing you...

That's an api miss-use. Your `Document` is destroyed after each iteration same as its allocator with allocated values. Resulting `std::vector` contains invalid `Value`s with dangling reference to de-allocated memory. Consider...

I recommend you to use `rapidjson::Document` array. I think it's preferred way to just store array of values: ```cpp rapidjson::Document array(rapidjson::kArrayType); array.PushBack(..., array.GetAllocator()); ```

> but I don't really know why No soname for Darwin, but there's a similar feature "Install name". It was used by you as a workaround but it's Darwin-only, so...

By the way, any thought about to add optional CMake support? I'd be happy to help with that.