operator[] returns by value

[feazeyu]

In my opinion, this behavior is very counterintuitive and the operator[] should return a reference instead, or it should be explicitly stated that jValue objects are intended as readonly.

As of now, the methods to modify jValue objects are public, thus semantically telling us that we should be able to modify them, however once you try to modify any values, you run into the issue of not being able to get a pointer to it.

Please fix the semantics or make the return be a reference instead of a value

[amir-s]

Hello @feazeyu
You are right! When I wrote this I implicitly assumed this snippet is to parse the json, not to create them. So it was designed to be "read-only" but it is not implemented / documented as such perhaps.

Would you be so kind and open a pull request? I'd gladly review and merge.