help rebasing

1.4.2 < 1.4.1

* Bug-fix for ValueIterator::operator-() (issue #169)

Merge pull request #172 from cdunn2001/master

Fix bug in ValueIteratorBase::operator-

Fixes #169.

reverse sense for CPPTL too

Fix bug in ValueIteratorBase::operator-

IteratorTest

rules for signing and doc-generation

1.4.1 <- 1.4.0

Merge pull request #167 from cdunn2001/fail-if-extra

Add `failIfExtra` feature to `CharReaderBuilder`.

failIfExtra

1. failing regression tests, from #164 and #107
2. implemented; tests pass
3. allow trailing comments

comments/minor typos

link web docs

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.