json_object: Avoid invalid free (and thus a segfault) when ref_count gets < 0

Merge pull request #386 from besser82/bugfix/aclocal_amflags

Makefile: Add ACLOCAL_AMFLAGS

Merge pull request #391 from rikardfalkeborn/fix-const-function-define

Fix non-GNUC define for JSON_C_CONST_FUNCTION

Fix non-GNUC define for JSON_C_CONST_FUNCTION

Makefile: Add ACLOCAL_AMFLAGS

This is recommended by the libtool manual.

Apply gcc's "const" attribute to the json_c_object_sizeof() function as an optimizer hint.  Also, rename that function from json_object_sizeof().

Merge pull request #388 from besser82/enhancement/json_object_sizeof

json_object: Add size_t json_object_sizeof()

Merge pull request #387 from LeSpocky/doc

doc: Use other doxygen feature to specify mainpage

json_object: Add size_t json_object_sizeof()

doc: Use other doxygen feature to specify mainpage

Previously a special tag was added to README.md to make this the
main page in Doxygen generated docs. When viewing the README on the
GitHub page this tag was not hidden.

After upgrading to Doxygen 1.8.8 (and above) in changeset
219025727dd7d981cf2fea7f2095570924e7bae7 a new Doxygen feature can be
used to specify the main page: Since release 1.8.3 Doxygen has a special
option to set a markdown file as main page. When using this option we
can drop the tag, making the README rendered by GitHub nice to look at.

Fixes: ae66b243690871f2daea5ae4b2a4669081d7c556

Remove the previously deprecated lh_table_lookup() function, in favor of lh_table_lookup_ex().

Remove the previously deprecated bits.h and the lh_abort() function.

Drop mention of README-WIN32.html in the release checklist.

Update the master branch to version 0.13.99

Tweak the release checklist slightly to fix problems noticed during this past release process.

Don't include private headers when installing.

Undeprecate the array_list, lh_table and printbuf typedefs, but move each to the corresponding header files.

Squash doxygen warnings, and make a few slight improvements to the docs.

Upgrade doxygen's Doxyfile to version 1.8.8

Add a brief overview of each file to the docs.

Adjust the description of json_object_iterator a bit.

Rearrange README.md a bit and make it work better in the doxygen generated docs.
Also, add a table of contents, and a brief section on using json-c with links to
the header files most likely to be useful.

Add a couple more issues to the list of those closed for 0.13

Deprecated a few typedefs, add docs on json_object_iter, move json_number_chars and json_hex-chars to json_object_private.h.

Cast to int64_t instead of long in test_int_add to fix the test on 32-bit platforms.

Add cast to int on tolower() to fix warnings about array subscripts.

Make sure to include the "*" on function pointer arguments to avoid a warnings from VS2015.  See also PR#384.

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