Have option to define RAPIDJSON_DEFAULT_ALLOCATOR

[Habbus]

Hi,

With the latest versions of rapidjson that I find on github, there is an option to manually define the memory allocator with a macro (this is from github):

// ... rapidjson/include/rapidjson/document.h

/*! \def RAPIDJSON_DEFAULT_ALLOCATOR
    \ingroup RAPIDJSON_CONFIG
    \brief Allows to choose default allocator.

    User can define this to use CrtAllocator or MemoryPoolAllocator.
*/
#ifndef RAPIDJSON_DEFAULT_ALLOCATOR
#define RAPIDJSON_DEFAULT_ALLOCATOR ::RAPIDJSON_NAMESPACE::MemoryPoolAllocator<::RAPIDJSON_NAMESPACE::CrtAllocator>
#endif

In the current version of rapidjson placed in the 3rdparty directory, this option does not seem to be there.
Would it be possible to have this as an option (maybe in the CMakeLists_Options.txt)? I have been experiencing some problems with the MemoryPoolAllocator, while the CrtAllocator works perfectly.

I tested this by manually changing the templated Allocators over here and here.
Also, it requires an update to the IMewssageConverter.

[Habbus]

@c-jimenez what do you think about this?

[c-jimenez]

Hi,

Sorry, I missed your question.

The rapidjson library in the 3rdparty folder is the latest available tag => quite old but stable version.
You can try to switch to a newer version and add the compile option : if it doesn't trigger regressions we can merge it in the develop branch.

Regards

[Habbus]

Thanks, I'll have a look at some point

[Habbus]

See #182