add real C++11 move semantics support about rapidjson

[pah]

Thanks for opening an explicit issue to track this. We have already discussed this in #91 and there is an (incomplete) patch available at pah/rapidjson-upstream@8005b55a.

Moving std::string instances to a Value won't work, though. The internal storage model is different between Values and std::strings and there is no way to move the raw data out of a std::string (even if we ignore the RapidJSON allocator support for now).

But at least temporary Value objects could be directly supported in AddMember and PushBack, which currently require an explicit call to .Move(), see Move semantics and temporary values.

C++11 move support would be useful for:

• Value
• Document, which implies
  • Allocator(s)
  • Stack(s)

[pah]

Hum, why did you close it?

[Kosta-Github]

I think everything is captured in your pah/rapidjson-upstream@8005b55 patch and the std::string move is not possible...

[pah]

Please reopen this issue to let us explicitly track the move support until we have completed and merged it, instead of some hidden discussion attached to a patch.

[pah]

After the merge of #173, I think we can close this one for now.