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.

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.

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.