Merge pull request #276 from bmyerz/master

make the unix cmake example work

Merge pull request #282 from keithkml/patch-1

Clarify which parts of README for users vs devs

Merge pull request #280 from mgorny/pkg-config-fix

Fix custom includedir & libdir substitution in pkg-config

Clarify which parts of README for users vs devs

When I arrived at the JsonCpp GitHub page, as an intermediate C++ developer, I could not figure out how to include JsonCpp into my project. The changes I propose to the README make this much clearer, and define a clear distinction between which instructions are for those developing and contributing to JsonCpp, and those who are just using it.

Fix custom includedir & libdir substitution in pkg-config

Do not prepend ${prefix} to substituted includedir & libdir
in the pkg-config file -- if the paths are overriden by user, CMake puts
absolute paths there (even if user specifies a relative path). Instead,
use the absolute path provided by CMake and appropriately default
LIBRARY_INSTALL_DIR & INCLUDE_INSTALL_DIR to absolute paths with
${CMAKE_INSTALL_PREFIX} prepended.

Fixes: https://github.com/open-source-parsers/jsoncpp/issues/279
Signed-off-by: Michał Górny <mgorny@gentoo.org>

Merge pull request #275 from stefan-it/stefan/cmake-generator-fix

[Documentation][Markdown] Use correct help option for cmake.

Merge pull request #277 from gogo40/master

fix compile error on android

But note that we do not have continuous integration testing for Android. This could break again.

`snprintf` drives me crazy. It should have been part of every C library 20 years ago.

fix compile error on android

make the unix make example work

...by setting the archive directory variable

[Documentation][Markdown] Use correct help option for cmake.

address compilation probs for C++ Builder

BORLANDC compiler strangeness. Thanks to:

* Dan Liu
* Victor Chen

close #269
close #252

another fix for BORLANDC

move ctors for BORLAND

Merge pull request #266 from cdunn2001/issue-252

Use unsigned for DuplicationPolicy, to fix a problem with "C++ Builder"
IDE.

Fixes #252.

Thanks to:

* Dan Liu -- http://blog.csdn.net/gzliudan/article/details/45264201)
* Victor Chen -- http://www.cppfans.com/sdk/json/jsoncpp.asp

a little test for issue 252, but does not fail for me

fix crash for "C++ Builder" IDE

http://blog.csdn.net/gzliudan/article/details/45264201

Merge pull request #265 from cdunn2001/valgrind

run valgrind in Travis CI

run valgrind in Travis CI

Because this runs apt-get, it will not work as-is for OSX. So when
we have OSX in Travis, we will have to wrap this somehow. See #250.

Closes #222.

Merge pull request #263 from cdunn2001/static-shared

Use standard **cmake** variables, to support superprojects better.

- `JSONCPP_LIB_BUILD_SHARED` -> `BUILD_SHARED_LIBS`
- `JSONCPP_LIB_BUILD_STATIC` -> `BUILD_STATIC_LIBS`

Use standard CMake variables - static/shared lib.

Replace JSONCPP_LIB_BUILD_SHARED => BUILD_SHARED_LIBS

Use standard CMake variables - static/shared lib.

Replace JSONCPP_LIB_BUILD_SHARED => BUILD_SHARED_LIBS

Use standard CMake variables - static/shared lib.

Replace JSONCPP_LIB_BUILD_SHARED => BUILD_SHARED_LIBS

Use standard CMake variables - static/shared lib.

Replaced JSONCPP_LIB_BUILD_SHARED => BUILD_SHARED_LIBS
Replaced JSONCPP_LIB_BUILD_STATIC => BUILD_STATIC_LIBS

Use standard CMake variables - static/shared lib.

Replaced JSONCPP_LIB_BUILD_SHARED => BUILD_SHARED_LIBS

Use standard CMake variables - static/shared lib.

Replaced JSONCPP_LIB_BUILD_SHARED => BUILD_SHARED_LIBS
Moved flag JSON_DLL to line no 8.

Use standard CMake variables - static/shared lib.

Replaced JSONCPP_LIB_BUILD_SHARED => BUILD_SHARED_LIBS
Moved definition DJSON_DLL to line 11.

Use standard CMake variables - static/shared lib.

Replace JSONCPP_LIB_BUILD_SHARED => BUILD_SHARED_LIBS
Replace JSONCPP_LIB_BUILD_STATIC => BUILD_STATIC_LIBS
Removed workaround  https://github.com/open-source-parsers/jsoncpp/issues/51
Removed OPTION for shared/static in this file.

Use standard CMake variables - static/shared lib.

Currently JSONCPP_LIB_BUILD_SHARED variable is used as option to build static/shared libraries.
The current patch uses standard CMake variables for this.
Such a workaround is done in https://github.com/open-source-parsers/jsoncpp/issues/51
Current patch will make it generic.

fix leak in unit-tests

swap docs for default vs. strictMode

Merge pull request #244 from cdunn2001/appveyor

New `appveyor.yml`: All tests pass, in both Appveyor and Travis!

Henceforth, GitHub will run both for any pull-request, so this file will be needed in the `0.y.z` branch too.

maybe fix an error

add a comment, to force a build

appveyor deploy init

fixed version on appveyor build

appveyor multiple platforms

appveyor.yml

prefer std::string for setComment()

in case of embedded nulls

1.6.2 <- 1.6.1

Fix UTF-8 for old (deprecated) Writers.

* Do not truncate at embedded zeroes.

Merge pull request #241 from cdunn2001/fix-more-utf8

support UTF-8 (specifically, embedded zeroes) in old Writers

support UTF-8 in old Writers

We had already fixed Value to hold UTF-8 properly, but only the newer
StreamWriter was writing UTF-8 properly.

Old FasterWriter etc. were using asCString() instead of asString() in
Value::writeValue().

Hopefully this change does not break any existing code. Seems unlikely.

issue #240

Merge pull request #239 from sbc100/copyright

Add copyright information to .py files

Add copyright information to .py files

This change adds explicit copyright information too python
files files.  The copyright year used in each case is the
date of the first git commit of each file.

The goal is to allow jsoncpp to be integrated into the
chromium source tree which requires license information in
each source file.

fixes #234

1.6.1 <- 1.6.0

Merge pull request #232 from cdunn2001/fix-snprintf

Fix snprintf

Well, it passes Travis. But when we have time, we should clean up how snprintf is used in both reader and writer.

use std::snprintf for C++11

Revert "Use std namespace for snprintf."

This reverts commit 1c58876185d2a4ed87dac4a54b82f607e74f55fd.

std::snprintf() is only available in C++11, which is not provided by
all compilers. Since the C library snprintf() can easily be used as a
replacement on Linux systems, this patch changes jsoncpp to use the C
library snprintf() instead of C++11 std::snprintf(), fixing the build error
below:

    src/lib_json/json_writer.cpp:33:18: error: 'snprintf' is not a member of 'std'

See #231, #224, and #218.

Merge pull request #225 from selaselah/master

fix find_program() bug: no result in not-win sys

fix find_program() bug: no result in not-win sys

Merge pull request #221 from btolfa/forgotten-virtual-dtor

Added forgotten virtual dtor for `Json::CharReader::Factory`.

(Without this, the destructor of the derived `CharReaderBuilder` would not be called, which is a small memory leak.)

Update reader.h

Merge pull request #219 from cdunn2001/c-std-headers

Close #218. Fix #214.

Use std namespace for snprintf.

Constrain MSVC _isfinite to before 2013, remove duplicate includes.

Fix isfinite for MSVC.

Use C++ standard headers.

Change exception data member

from "reference to string" to "string" (Resolves the most serious part of issue #216)

prefer ValueIterator::name() to ::memberName()

in case of embedded nulls

Merge pull request #212 from cdunn2001/macro-deprec

close #210

Make preprocessor query robust against older gcc versions

Provide JSONCPP_DEPRECATED definitions for clang and gcc

1.6.0

comments

Merge pull request #211 from cdunn2001/except

* Add Json::Exception and derivatives.
* Clarify when exceptions are thrown, to avoid crashes caused by malicious input.
* Use our own type (derived fro std::exception) so they are trappable.

use Json::RuntimeError

use Json::LogicError in macros

Runtime/LogicError and throwers

base Json::Exception

clarify errors

* use macros for logic errors, not input errors
* throw on parsing failure in `operator>>()`, not assert
* throw on malloc, not assert

delete debug code from test

require length

Ugh! I meant to do this long ago. It would have caught my blunder.

1.5.4 <- 1.5.3

important bug-fix (thx to datadiode@)

Merge pull request #207 from cdunn2001/fix_CZString_copy_constructor

Fix czstring copy constructor

drop unused CString ctor case

`Value::CZString::CZString(char const* str, unsigned length, DuplicationPolicy allocate)` with `allocate == duplicate` does not happen.

Trivial fixes in CZString constructors.

fix ValueTest/zeroes*

old-style enum namespacing

Merge pull request #205 from open-source-parsers/reject-dup-keys

[Shekhar (shakers007) wrote](https://sourceforge.net/p/jsoncpp/bugs/22/):

> As per RFC4627 (section 2.2), names within an object should be unique. When using JSONCPP's strict mode, parsing such an object should fail.

rejectDupKeys

add rejectDupKeys feature - not yet impld

test for repeated key in strictMode

https://sourceforge.net/p/jsoncpp/bugs/22/

change str_ for cross-compilation

https://sourceforge.net/p/jsoncpp/bugs/59/

clarify operator=

test assignment over-writes comments, but swapPayload() does not

test commentBefore

for issue #203

clarify which versions work with old compilers

1.5.3 <- 1.5.2

generate both version.h and version from CMakelists.txt

This forces consistency, since they will be re-generated whenever
a git operation alters CMakelists.txt. They are still in the repo
because users might not actually run cmake.

had trouble finding Python on Windows

With this change, `make jsoncpp_check` will still fail if Python
is missing, so our CI tests are unaffected.

Merge pull request #202 from open-source-parsers/get-with-zero

`Value::get(key, default)` with zero

fix get() for embedded zeroes in key

This method had been overlooked.

test get(key, default)

Merge pull request #201 from open-source-parsers/vs

* Copy .dll for running unit-tests in VisualStudio.
* Stop using `do{}while(0)` idiom b/c VisualStudio warns.
* Fix a warning in a test.

drop `do{}while(0)` idiom

Rationale:
  * http://stackoverflow.com/questions/154136/do-while-and-if-else-statements-in-c-c-macros/154138#154138

But Visual Studio issues a warning: `warning C4127: conditional expression is constant`
  * http://stackoverflow.com/questions/1946445/c-c-how-to-use-the-do-while0-construct-without-compiler-warnings-like-c412

copy .dll for unit-test

Fix 2nd problem in issue #200.
* http://stackoverflow.com/questions/10671916/how-to-copy-dll-files-into-the-same-folder-as-the-executable-using-cmake

Q: What about the Python tests?
A: They are not normally run in Visual Studio. If desired, one can set PATH.

maybe address warning

    cmake -DCMAKE_BUILD_TYPE=debug -DJSONCPP_LIB_BUILD_STATIC=OFF
-DJSONCPP_LIB_BUILD_SHARED=ON -G "Visual Studio 10" ../..

`potentially uninitialized local variable 'dist' (line 2212 of
test_lib_json/main.cpp)`

maybe fix DLL symbols for VS

1.5.2 <- 1.5.1

* Fixed compile error for VS2013.
* Added `operator[]` to Builders. Recalling previous minor versions for
  bug-fixes.

Merge pull request #199 from open-source-parsers/set-builders

Setting builders

fix example in docs