stop calling validate() in newReader/Writer()

By not calling validate(), we can add
non-invasive features which will be simply ignored when user-code
is compiled against an old version. That way, we can often
avoid a minor version-bump.

The user can call validate() himself if he prefers that behavior.

Merge pull request #166 from cdunn2001/stackLimit

Fixes #88 and #56.

limit stackDepth for old (deprecated) Json::Reader too

This is an improper solution. If multiple Readers exist,
then the effect stackLimit is reduced because of side-effects.
But our options are limited. We need to address the security
hole without breaking binary-compatibility.

However, this is not likely to cause any practical problems because:

* Anyone using `operator>>(istream, Json::Value)` will be using the
new code already
* Multiple Readers are uncommon.
* The stackLimit is quite high.
* Deeply nested JSON probably would have hit the system limits anyway.

limit stackDepth

stackLimit

stackLimit_

test stackLimit

Old -> Our

1st StreamWriterTest

clarify comments

And throw instead of return null for invalid settings.

Merge pull request #165 from cdunn2001/master

Remove some experimental classes that are not needed for 1.4.0. This also helps 0.8.0 binary compatibility with 0.6.0-rc2.

drop experimental OldCompressingStreamWriterBuilder

enableYAMLCompatibility and dropNullPlaceholders for StreamWriterBuilder

nullRef, since we had to add that kludge to 0.8.0

copy Features to OldFeatures

passes

copy Reader impl to OldReader

renames for OldReader

OldReader copied from Reader

Merge pull request #163 from cdunn2001/master

Reimplement the new Builders.

Issue #131.

mv CommentStyle to .cpp

set output stream in write(), not in builder

<stdexcept>

add all features

remark defaults via doxygen snippet

Builder::settings_

We use Json::Value to configure the builders so we can maintain
binary-compatibility easily.

update docs, writeString()

Merge pull request #162 from cdunn2001/master

Deprecate the new Builders.

deprecate current Builders

deprecate Builders

see issue #131

deprecate old Reader; separate Advanced Usage section

--in=doc/web_doxyfile.in

copy doxyfile.in

drop an old impl

simplify basic docs

clarify Builders

put version into docs

Merge pull request #161 from cdunn2001/master

CharReader/Builder

I guess we should but the patch-level version. We will set the version properly soon...

CharReader/Builder

* CharReaderBuilder is similar to StreamWriterBuilder.
* use rdbuf(), since getline(string) is not required to handle EOF as delimiter

Merge pull request #160 from cdunn2001/master

rm unique_ptr<>/shared_ptr<>, for pre-C++11

rm unique_ptr<>/shared_ptr<>, for pre-C++11

Merge pull request #158 from cdunn2001/travis-with-cmake-package

JSONCPP_WITH_CMAKE_PACKAGE in Travis

I guess we don't really need to shared and static separately either. Saves a little time, maybe?

JSONCPP_WITH_CMAKE_PACKAGE in Travis

Merge pull request #156 from cdunn2001/with-cmake-package

fix JSONCPP_WITH_CMAKE_PACKAGE #155

fix JSONCPP_WITH_CMAKE_PACKAGE #155

mv JSONCPP_WITH_CMAKE_PACKAGE ahead of INSTALL def.

drop nullptr

See issue #153.

drop scoped enum, for pre-C++11 compatibility

added cmake option to build static and shared libraries at once

See #147 and #149.

docs

Merge pull request #145 from cdunn2001/simplify-builder

Simplify builder

gcc-4.6 (Travis CI) does not support

docs

fix doc

make StreamWriterBuilder concrete

Move old FastWriter stuff out of new Builder

OldCompressingStreamWriterBuilder

improve docs

including `writeString()`

Merge pull request #132 from cdunn2001/builder

StreamWriter::Builder

Deprecate old Writers, but include them in tests.

This should still be binary-compatible with 1.3.0.

deprecate old Writers

also, use withers instead of setters, and update docs

implement strange setting from FastWriter

implement CommentStyle::None/indentation_==""

default \t indentation, All comments

incorporate 'proper newlines for comments' into new StreamWriter

clarify CommentStyle

drop StreamWriterBuilderFactory, for now

copy fixes from StyledStreamWriter

test BuiltStyledStreamWriter too

const stuff

non-const write

copied impl of StyledStreamWriter

fix comment

use new BuiltStyledStreamWriter in operator<<()

setIndentation()

StreamWriter::Builder

StreamWriter

update docs

add back space before trailing comment

Merge pull request #144 from cdunn2001/proper-comment-lfs

proper newlines for comments

This alters `StyledStreamWriter`, but not `StyledWriter`.

proper newlines for comments

The logic is still messy, but it seems to work.

Merge pull request #143 from cdunn2001/rm-trailing-newlines

rm trailing newlines for *all* comments

rm trailing newlines for *all* comments

This will make it easier to fix newlines consistently.

Merge pull request #141 from cdunn2001/set-comment

Fix a border case which causes Value::CommentInfo::setComment() to crash

Fix a border case which causes Value::CommentInfo::setComment() to crash

re: pull #140

TravisCI gcc-4.6 does not yet support -Wpedantic

Merge pull request #139 from cdunn2001/some-python-changes

Some python changes.

* Better messaging.
* Make `doxybuild.py` work with python3.4

README.md

make doxybuild.py work with python3.4

Doxybuild: Error message if doxygen not found

This patch introduces a better error message.

See discussion at pull #129.

update doxbuild.py

update TarFile usage

rm extra whitespace in python, per PEP8

fix issue #90

We are static-casting to U, so we really have no reason to use
references.

However, if this comes up again, try applying -ffloat-store to
the target executable, per
    https://github.com/open-source-parsers/jsoncpp/issues/90

fix gcc warnings

Merge pull request #138 from cdunn2001/fix-103

Fix #103.

fix test_comment_00 for #103

add test_comment_00

one-element array with comment, for issue #103

Merge pull request #137 from cdunn2001/avoid-extra-newline

Avoid extra newline

1.4.0 <- 1.3.0

Minor version bump, but we will wait for a few more commits this time
before tagging the release.

avoid extra newlines in StyledStreamWriter

Add indented_ as a bitfield. (Verified that sizeof(StyledStreamWriter)
remains 96 for binary compatibility. But the new symbol requires a minor
version-bump.)

Merge pull request #136 from cdunn2001/test-both-styled-writers

Test both styled writers

Not only does this now test StyledStreamWriter the same way as StyledWriter, but it also makes the former work more like the latter, indenting separate lines of a comment before a value. Might break some user tests (as `operator<<()` uses `StyledStreamWriter`) but basically a harmless improvement.

All tests pass.