Merge back

v_ has a value assigned to it as part of ParseNext() which happens in
the constructor, but that’s not soon enough for GCC

GCC and Clang cannot agree on what constitutes a good switch statement.

Issue 889

Fix clang -Wswitch-enum warnings.
Made NextArrayValue() more robust—now handles error state correctly,
will enter error state if an unexpected state is reached.
Made separate states for each value type to simplify getters.
Simplified implementation of skipping arrays and objects.
Skipping an object now works whether you’ve retrieved the key or not.

It did not support char arrays.

Lookahead Parser

Fix -Werror=effc++ errors with GNU 6.3.1

Fix "'MyHandler::type’ should be initialized in the member
initialization list [-Werror=effc++]" errors.

https://github.com/miloyip/rapidjson/issues/874

Fix "'MyHandler::type’ should be initialized in the member
initialization list [-Werror=effc++]" errors.

https://github.com/miloyip/rapidjson/issues/874

Use native strlen

Disambiguate by putting the ENABLEIF on the return value instead of in
the argument list.

Assert in case users attempt to pass a char array to String() or Key()
that is not null terminated; that is not the intended use of the API.
Null terminate your string buffers.

Same fix as previous commit, to prettywriter

No strlen call needs to be made when templates can auto-deduce the
string length. No strlen = faster!

Unfortunately this needs a touch of  SFINAE to allow multiple overrides
to coexist cleanly.

Remove executable bit

Remove the executable bit for various .json files

No-op blank line

Token-by-token parser documentation

Compilers generally provide a much smarter strlen than ours.

Make example code var names match API above for consistency

Writer EndObject missing-value fix