WSoptics
WSoptics
As discussed in another pull request, this is problematic since `std::variant` allows for the same type to exist multiple times. So this makes no sense in general.
What's the point of not using a `std::string` to begin with? I think this pull request should be rejected since: * It changes global behavior based on a preprocessor directive....
> In my use case I'm using fixed size char arrays to represent strings because std::string allocates heap. I guess it is a niche use case and will be deprecated...
It might be a good idea to make this an option of `JSONArchive` instead of altering its default behavior?