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.

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

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.

Issue #308: improve the build instructions in README.md to include the exact commands to run for installing prerequisites, as mentioned earlier on Issue #308.

Issue #332: fix a long-standing bug in array_list_put_idx() where it would attempt to free previously free'd entries due to not checking the current array length.
Add a test that triggers the problem to ensure it stays fixed.

Merge pull request #312 from Tailmon/master

Fix CMake Build process improved for MinGW and MSYS2

Merge pull request #319 from Dashlane/visual-studio-build

Windows: Fix dynamic library build with Visual Studio

Merge pull request #329 from commodo/rename-static-lib

build,cmake: build,cmake: rename libjson-c-static.a to libjson-c.a

Merge pull request #330 from commodo/symlink_some_tests

tests: symlink basic tests to a single file that has the common code

Merge pull request #333 from besser82/bugfix/obsolete_macros_autotools

Replace obsolete AM_PROG_LIBTOOL

Replace obsolete AM_PROG_LIBTOOL

tests: symlink basic tests to a single file that has the common code

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

build,cmake: set C_STANDARD 99 property to libjson-c-static

Apply the same property as to json-c.

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

build,cmake: rename libjson-c-static.a to libjson-c.a

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

Merge pull request #321 from commodo/fix-cmake-vasprintf

build,cmake: fix vasprintf implicit definition and generate both static & shared libs

Issue #161: add a json_object_to_fd() function.

Fix test_double_serializer expected output.

Add a json_c_set_serialization_double_format() function to set the *library-wide* format for how doubles are written to a serialized JSON output.

Make _set_last_err() non-static so it can be used outside of json_util.c

Merge pull request #325 from rouault/fix_stack_overflow_in_json_object_double_to_json_string_format

Fix stack buffer overflow in json_object_double_to_json_string_format()

Fix stack buffer overflow in json_object_double_to_json_string_format()

Issue originally found in the json-c 0.11 internal copy in GDAL but also found
in latest git version.

If doing things like
json_object* obj = json_object_new_double(1e300);
json_object_set_serializer(obj, json_object_double_to_json_string, "%f", NULL);
json_object_to_json_string(obj)

    size = snprintf(buf, sizeof(buf),
        format ? format :
          (modf(jso->o.c_double, &dummy) == 0) ? "%.17g.0" : "%.17g",
          jso->o.c_double);
will return a value greater than 128 since at least 300 characters are needed.
This value is then passed to printbuf_memappend(pb, buf, size); that tries to
read size bytes in buf.

So we should clamp size to sizeof(buf). And on Windows, _snprintf() returns -1
in that situation, so deal also with this case.

Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=1682
Credit to OSS-Fuzz

.gitignore: add cmake generated files

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

build,cmake: generate both static and shared libjson

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

build,cmake: add _GNU_SOURCE to CFLAGS

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

Fix library build with Visual Studio

Fixed linkhash.c for MinGW

Added #ifdef conditions for MinGW because this compiler doesn't define
gcc atomic builtins, so __sync_val_compare_and_swap was not found
neither any of any __GCC_HAVE_SYNC_COMPARE_AND_SWAP_* definition

Fix CMake Build process for MinGW and MSYS2

This fix includes some fixes for config.h.win32 that was generating
some compiling errors on MinGW64 and also fix CMakeLists.txt in order to
give better support to MinGW and MSYS2

Merge pull request #311 from jj1118/master

Fix error C3688 when compiling on Visual Studio 2015

Fix error C3688 when compiling on Visual Studio 2015: invalid literal suffix 'PRId64'; literal operator or literal operator template 'operator ""PRId64' not found

Merge pull request #310 from marxin/fix-gcc7-warnings

Add FALLTHRU comment to handle GCC7 warnings.

Add FALLTHRU comment to handle GCC7 warnings.

Merge pull request #305 from alexanderst/master

Fix compilation without C-99 option

Fix compilation without C-99 option

Add a few more cases to test_parse to provide some examples of how parsing works; should help address the questions raised in Issue #302.

Merge pull request #303 from yogo1212/tokener_h

fix doc in tokener header file

fix doc in tokener header file

Issue #300: Cast size_t values to int instead of unsigned long long to work around broken behavior of MinGW.

Issue #300: Some platforms, such as MinGW, don't have ETXTBSY and ENOTBLK, so skip those there.

Issue #300: Use %p instead of %lx in test_util_file since some platforms' pointers are larger than "unsigned long".
Also, there's no need to worry about output consistency here, since it'll be
 different anyway due to different pointer values.

Add cast to int in calls to isdigit() since some platforms complain about char being used as an array index.

Merge pull request #299 from qlyoung/perf-improvements

Improve json_object -> string performance

Restore sprintbuf(), add macro for string literals

Hawciz pointed out that the previous commit modifies the public
interface of printbuf. Per his suggestion, sprintbuf() was restored
and a new pair of macros was added that wraps printbuf_memappend().

Using a wrapper macro instead of modifying sprintbuf() also reduces
function call overhead, bringing total performance gains to
approximately 400%.

Use strdup instead of strndup in test1.c, there's no need for the latter because json_object_to_json_string_length() will always return a properly sized string.

Improve json_object -> string performance

Removes variadic prints for ~3x performance improvement.

Issue #295: also check if size_t is the size of long long, to help support 64-bit Windows platforms.

Ignore tests/test_float

Change a memcpy that should be a memmove within json_pointer_get, and fix memory leaks in of one the json_pointer tests.

Note some minimum versions needed for autoconf, etc...

Issue#292: bump the required version of configure listed in configure.ac so a more useful error message is generated, rather than failing later because the extra macros from ./autoconf-archive/m4/ didn't get loaded.

Merge pull request #289 from jgerhards/i-278

bugfix: floating point representaion without fractional part

testbench: add test for floating point representation

bugfix: floating point representaion without fractional part

closes https://github.com/json-c/json-c/issues/278

Merge pull request #287 from commodo/json_pointer_va_args

json_pointer: extend setter & getter with printf() style arguments

vasprintf_compat.h: spin-off this compat header ; use math_compat.h as template

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

strdup_compat.h: re-spin this compat header ; use math_compat.h as template

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

test_json_pointer: update test with a few printf variants

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

json_pointer: add json_pointer_getf/setf() function variants

These include support for printf() style args for path.

Adds support for calling with 'json_pointer_getf(obj, &res, "/foo/%d/%s", 0, bar)'
style args.

Makes it easier for doing more dynamic stuff/magic, without
needing to use vasprintf() externally.

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

json_pointer.c: fix whitespace

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

Merge pull request #285 from ebassi/revert-strdup-compat

Revert "compat/strdup.h: move common compat check for strdup() to own…

Revert "compat/strdup.h: move common compat check for strdup() to own file"

This reverts commit aaba8c1080533133329086e47f4f45981dfdd1ee.

This commit breaks builddir != srcdir build, but, more importantly, it
also adds a dependency on a header, "config.h", which is not installed
and it's supposed to be private — since it's generated at configuration
time and it's not meant to be used by projects compiling against a
library.

Merge pull request #283 from commodo/json_pointer

json_pointer: add first revision based on RFC 6901

tests: add test_json_pointer test

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