Merge pull request #384 from ssrlive/patch-1

Fix a VS 2015 compiler warning.

Update json_object.c

Fix a VS 2015 compiler warning.

In VS 2015, the warning text is "warning C4550: expression evaluates to a function which is missing an argument list".

Add a change log entry for the upcoming 0.13 release.

Clean up *.vg.out files too to "make distcheck" works.

In json_object_deep_copy(), copy over _userdata, at least for json_type_string's with the default serializer set, and provide a way for people using custom serializers to provide a custom shallow_copy method.

Always build the json_object_deep_copy() code, but conditionalize running it on a --benchmark command line option.

Rename _set_last_err() to _json_c_set_last_err().

Merge branch 'commodo-json_deep_copy'

Merge branch 'json_deep_copy' of https://github.com/commodo/json-c into commodo-json_deep_copy

Rename json_object_add_int() to json_object_int_inc() and eliminate the "int64" variant since we store 64-bit values internally anyway.

tests: add test_deep_copy test

Seems to perform better than outputting to string
and re-parsing it.

BENCHMARK - 1000000 iterations of 'dst2 = json_tokener_parse(json_object_get_string(src2))' took 20 seconds
BENCHMARK - 1000000 iterations of 'dst2 = json_tokener_parse(json_object_get_string(src2))' took 7 seconds

It should make a difference on embedded systems.
The test was performed on a i5 desktop CPU [~3.5 years of age].

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

json_object: implement json_object_deep_copy()

Because doing `json_tokener_parse(json_object_get_string(src))`
feels sloppy, dirty, and makes me want to cry at night
sometimes.

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

Merge pull request #363 from jvijtiuk/integer_addition

Issue #338, add json_object_add_int functions

Update test_parse.expected to match the code.

Rewrite the json_tokener_state_inf handling in json_tokener to be simpler, and avoid needless copying of the input into a temporary buffer.

Add a few more tests to the partial parse to cover bytes after "Infinity".

Save proper expected output for test_parse from the previous commit.

Fix parsing of "-Infinity" when the "-" and "Infinity" are split across multiple calls.
Add several additional test cases for partial parsing of infinity values.

Issue #371: fix parsing of "-Infinity" (although in a somewhat different location than PR#372 used), and add a case to test_parse to check for this.

Add a few extra cases to test_parse to show how trailing bytes are handled, especially for parsing "Infinity".

Rewrite test_basic_parse() to factor out a single_basic_parse() to avoid lots of duplicate boiler plate code.
Also, emit the input string in the output so it's easier to see what's going on.

On VS 2013 and newer, actually use strtoll instead of redefining it to _strtoi64.

Merge pull request #373 from commodo/fix-appveyor-build

build: fix build on appveyor CI

Merge pull request #381 from busterb/fix-makedist

Fix makedist

add/remove missing distribution files

ignore more autoconf goo

json_util: define `strtoll` as _strtoi64 for MSVC

Got the idea from this blog post:
  http://www.enchantedage.com/node/231

Simple & concise stuff :)

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

strerror_override: re-organize strerror_override.h

Always include <string.h> before _json_c_strerror() definition.
Should fix linker issues on MSVC.

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

Remove out of date win32 build information, and mention the need to use cmake there.

Fix definition of NELEM in the tests.

Issue #370: work around Clang-on-windows oddities by rearranging ifdefs.

Issue #369: note some useful settings when building for Android.

Use cmake to create the json-c.pc file, as suggested by PR#362.

Issue #173, follow up to using strtoll to allow this to work on older Windows environments: Use cmake to generate config.h from config.h.win32, including checking for strtoll with cmake, or fall back to _strtoi64 for older MSVC's.
Also, add a few missing files to the list of sources to build.

The new cmake-generated VS builds now generate a dynamic .dll instead of a .lib

For appveyor builds, json_config.h is actually generated in the "include" directory.

Split the list of cmake headers into public and private ones, and only install the public ones.

Add a "cmake ." line to the appveyor build file to try to get it working again.

Add json_object_add_int functions

Merge pull request #361 from schwehr/int64

Fix double to int cast overflow in json_object_get_int64.

Fix double to int cast overflow in json_object_get_int64.

Found with autofuzz in GDAL

Merge pull request #360 from jasonbking/master

Use strtoll() to parse ints

Issue#353: attempt to fix or ignore a few -Wdocumentation messages from Clang.

Issue#353: mark lh_abort as actually deprecated, not just in the docs.

Add long long 64-bit check

Use strtoll to parse ints

Remove the Visual Studio project files as they were out of date, and should now be generated by using cmake.

Issue #359: Use consistent spacing in CMakeLists.txt

Issue #359: Don't duplicate lh_get_hash, just omit the "inline" for VS2010.

Merge pull request #359 from Haffon/api-0.12

update CMakeLists.txt for compile with visual studio at least 2010

set JSON_C_HEADERS full file name

if compile with vs2015, enable the "static inline" declare.

rollback api to 0.12

Merge pull request #2 from json-c/master

merge upstream

Undo a bit of 2d1da5ab: handle per-thread formats for double serialization, even if --enable-threading wasn't specified.

Issue #173: since some sscanf implementations return 0 for non-zero inputs, directly check for "0" in the input.

Add a --enable-threading configure option, and only use the (slower) __sync_add_and_fetch()/__sync_sub_and_fetch() function when it is specified.

Use AC_CONFIG_MACRO_DIRS to specify path to the ax macros instead of passing -I to autoreconf in autogen.sh.

Explicitly check for GCC's atomic functions instead of depending on the __GNUC__ define.
Add a comment mentioning the limitation even though the _ref_count value is hanled atomically.

Merge branch 'pull-211'
This is EmielBruijntes' "Atomic updates for the refcount"

Issue #351: don't redefine SIZE_T_MAX if it's already defined.

Issue #349: none of automake's clean targets are suite for really cleaning up everything, so add a local "really-clean" target that does so.

Merge pull request #1 from json-c/master

want to merge the original repository

PR#331: for Visual Studio, use a snprintf/vsnprintf wrapper that ensures the string is terminated.

1.make it can been compiled with Visual Studio 2010
2.replace json_object_get/put API with json_object_retain/release, as they operate the reference counter, and confused with array_list_get/put_idx.
3.replace array_list_get/put_idx API with array_list_get/insert to make them more clear to use.

Merge pull request #346 from schwehr/get_int

Clamp double to int32 when narrowing in json_object_get_int.

Merge pull request #345 from MrAnno/fix-make-dist

Fix make dist and make distcheck

Merge pull request #344 from fastogt/master

Fix Mingw build

Clamp double to int32 when narrowing in json_object_get_int.

Avoids undefined behavior.  Found by autofuzz.

Fix 'make distcheck'

Signed-off-by: László Várady <laszlo.varady@balabit.com>

Fix 'make dist'

EXTRA_DIST copies the listed directories/files from the _source_ directory
into the distribution.

Since the doc directory does not exist after running autogen + configure
+ make dist, the distribution tarball generation fails.

Note that the dist-hook rule below operates on 'distdir', not on the source
directory where EXTRA_DIST expects the existence of the doc folder.

In summary, even if I removed 'doc' from EXTRA_DIST, the dist tarball will
always contain the documentation (due to the dist-hook rule).

Signed-off-by: László Várady <laszlo.varady@balabit.com>

Fix Mingw build

Fix parsing doubles for mingw

PR #336: fix to previous change, be sure to include string.h when we're using the real strerror.

PR #336: Fix typo in defining STRERROR_OVERRIDE_IMPL

Merge pull request #340 from commodo/fix-appveyor-build

strerror_override: add extern "C" and JSON_EXPORT specifiers for Visual C++ compilers

build,travis: drop `-enable-strerror-override` argument (no longer exists)

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

strerror_override: add extern "C" and JSON_EXPORT specifiers for Visual C++ compilers

Fixes build on AppVeyor.

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

PR #336: since we can't use function overriding (due to problems with it on
OSX) always include the _json_c_strerror function but only enable it with a flag
 during tests.

PR #336: since we can't use function overriding (due to problems with it on OSX) always include the _json_c_strerror function but only enable it with a flag during tests.

Allow USE_VALGRIND to be set to anything starting with 0, N or n to disable valgrind during tests.

Merge pull request #336 from commodo/fix-tests

tests: fix tests in travis-ci.org

build,travis: enable strerror override option in build

To get consistent output between Linux & OS X.

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

build: make `strerror()` override-able

If we want to override `strerror()` in libjson-c
to make tests consistent across platforms, we
need to do it build-wide as configure/build
option.

Apple linkers make it really hard to override functions
at link-time, and this seems to be locked down on travis-ci.org
[ for security reasons I assume ].
While I got it to work locally, it did not work
when running on travis.

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

configure.ac: check for `uselocale` function only on Linux platforms

On Apple this seems to fail the `test_locale` test,
which would imply that the `uselocale` function
does not behave as expected.

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

tests: fix leak in `test_util_file` ; found by cppcheck

Which now seems to fail the build.

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

travis,tests: run cppcheck only if it exists

ugh... seems cppcheck is not packaged for OS X
And `set -e` exposes this.

And also `cppcheck` seems to exit with non-zero
exit codes by default [even if errs found].

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

tests: compress test_utile_file with test_basic

More code compression/de-duplication.

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

tests: add `set -e` specifier to bail early on build run

Seems that test1 is failing, but travis is not catching it.
Likely, this is because the `cppcheck` returns success
and we need to bail on the `make check` step.

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

Merge pull request #335 from commodo/build_status_travis

README.md: show build status tag from travis-ci.org

README.md: show build status tag from travis & appveyor.

Looks nice to see a `Build Passing` tag when you
scroll to the README.

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

Move a variable declaration to the start of the block to work better with older compilers.

Remove a spurious comma in configure.ac

Updated expected test1 output which should have been included in commit fd9b3b2.

Comment out the warning about racy random seed initialization in lh_char_hash(), if you're on a platform where it'll be triggered it just makes it a pain to build.

Add extra casts to void * to squash some warning on certain systems (e.g. CentOS w/ gcc 4.1.2).

Fix bad usage of strncat introduces in 1a94c70.  Pointed out by @rouault in PR #331.

Eliminate static qualifiers on a couple local variables that were causing thread safety issues.  Suggested by @rouault in PR #331.

Reformat json_object_double_to_json_string_format() to have consistent spacing.