Issue #260: add a check to prevent trivial loops in the object tree, even though it is up to the callers to avoid doing so in more complex cases.

json_object_object_add_ex() should have shared code with json_object_object_add(), and been changed to return int at the same time the latter was.  Do that now.
Also correct a couple of calls to _to_json_string to check the return value.

Merge branch 'RyDroid-tests'

Merge branch 'tests' of https://github.com/RyDroid/json-c into RyDroid-tests

Conflicts:
tests/test_util_file.c

Merge pull request #251 from RyDroid/editorconfig

Adding a file for EditorConfig

Very minor changes to some tests

Adding a file for EditorConfig

Revert bogus change to json_object_get() made in f40b08d8 that caused it to always return NULL. (Issue #259)
Also undo NULL check in json_tokener_set_flags(): it's not at all valid toi
 try to set flags on a NULL pointer, and doing so should not be silently
 ignored.

Merge pull request #255 from RyDroid/minor-c2

Minor changes in C source code

Merge pull request #252 from RyDroid/minor-no-c

Very minor changes not related to C source code

Merge branch 'jobol-issue-165'

Merge branch 'issue-165' of https://github.com/jobol/json-c into jobol-issue-165

Conflicts:
tests/test_util_file.c

Merge pull request #253 from RyDroid/cppcheck-travis

Adding a test with cppcheck for Travis CI

Merge pull request #249 from RyDroid/readme

Improving README

Add a brief style guide.  Address Issue #257.

Merge pull request #250 from RyDroid/gitignore

Improving .gitignore

Improving .gitignore

Improving README

Adding a test with cppcheck for Travis CI

Minor changes in C source code

Very minor changes not related to C source code

Issue #246: Include xlocale.h too, to get locale_t defined, since not all OSes include that in locale.h

test: add  test of 'json_object_to_json_string_length'

The test is implied in test1.

The idea is to check that the returned lengths and strings
are identical to what is expected to return the already
tested function 'json_object_to_json_string_ext'.

Signed-off-by: José Bollo <jose.bollo@iot.bzh>

Issue #195: Actually call uselocale() in the new locale handling code in json_tokener.
Also, be sure the right locale_t is freed if we fail on the second uselocale.
Finally, fix test_locale so it *doesn't* use json_object_to_json_string as
 that will simple re-emit the original parsed string values.

Issue #195: use uselocale() instead of setlocale() in json_tokener to behave better in threaded environments.

Linux's %p format doesn't return "0x0" for NULL pointers, so switch to using %lx for consistency.

Issue #240: add arbitrary use of val's in foreach loops to squash warnings about unused variables.

Add method 'json_object_to_json_string_length'

This new method allows to also
get the length of the generated string.

Fix #165

Change-Id: Iea91404027f143ca3d29a4c58d7c07ae53556110
Signed-off-by: José Bollo <jose.bollo@iot.bzh>

Fix a compiling error

The function fstat isn't declared without this include
on debian strect/sid and Fedora-23

Change-Id: I660a32ff173dcba04674aed51ed855b4fa55ac67
Signed-off-by: José Bollo <jose.bollo@iot.bzh>

Merge pull request #239 from ploxiln/printbuf_empty

initialize null terminator of new printbuf

initialize null terminator of new printbuf

It's possible (e.g. by using json_object_from_file() on an empty file)
to get json-c to try to use a printbuf that has never had anything
written to it. Before this change, it could access a string that
should be length zero, but was never initialized, and could
theoretically have an unexpected string.

Fix absolute paths in test_util_file that vary depending on where it is run.

Issue #113: Add more files to EXTRA_DIST and a dist-hook to get distcheck to at least not fail.

Issue #113: add "new" files to appropriate variables in Makefile, and note the need to run "make distcheck" as part of the release process.
Fix issues with the test script so it works under distcheck too.

Merge pull request #238 from nagamalli9999/master

linkhash.c: optimised the table_free path

Issue #189: Eliminate use of MC_ERROR from json_util.c, and add a json_util_get_last_err() function to retrieve the error for those callers that care about it.
Add tests and descriptions for the functions in json_util.c

linkhash.h: removed redundant params from comments

linkhash.c: optimised the table_free path

Issue #236: Add -Wcast-qual and fix casts to retain constness.
To better distinguish between entry->k and entry->v being const within linkhash, but non-const outside, add lh_entry_v() and lh_entry_k() accessors.
Make lh_entry->k const.

Issue #175: disable the fast-and-loose code in hashlittle() when running with AddressSanitizer.

Mention that we're aiming to follow RFC 7159 now.
Cause README.md to show up in Doxygen output as the main page.

Merge pull request #232 from NeoRaider/size_t2

tests/tests1: fix printf format for size_t arguments

tests/tests1: fix printf format for size_t arguments

Change %d to %llu and add cast to unsigned long long for size_t arguments,
otherwise compilation will fail with errors like:

test1.c:70:15: error: format ‘%d’ expects argument of type ‘int’, but argument 2 has type ‘size_t {aka long unsigned int}’ [-Werror=format=]

%zu is avoided to stay compatible with old libc versions (like old Visual
Studio).

Copy over note about the 0.12.1 release to the master branch.

Check the __GCC_HAVE_SYNC_COMPARE_AND_SWAP_{2,4,8} defines to decide whether to use __sync_val_compare_and_swap(), as described at https://gcc.gnu.org/onlinedocs/cpp/Common-Predefined-Macros.html
Also, fix the types of the variables when building on Windows.
Also, should address issue #214.

Merge pull request #230 from ebassi/master

Use stdint.h to check for size_t size

Merge pull request #233 from NeoRaider/size_t3

Include stddef.h in json_object.h

Merge pull request #234 from NeoRaider/userdata

Add public API to use userdata independently of custom serializer

Remove reference to svn.metaparadigm.com

svn.metaparadigm.com no longer exists so it is not possible to use source from there.

Add public API to get and set userdata

Also, json_object_set_serializer is changed to respect the userdata
and user_delete parameters when to_string_func is NULL.

Make default double serializer ignore userdata again

The user might want to use the userdata for something different, so the
serializer should ignore it by default.

Explicitly setting the serializer to json_object_double_to_json_string will
still make it interpret the userdata as a format string.

Include stddef.h in json_object.h

It is needed for size_t.

Use stdint.h to check for size_t size

If we use json_inttypes.h we are going to fail when builddir != srcdir,
and since JSON_C_HAVE_INTTYPES_H is defined after the configure script
has run, including json_inttypes.h would be the equivalent on including
stdint.h anyway.

This fixes the build of json-c in the GNOME Continuous builder.

Issue #142: un-deprecate json_object_object_get(), but note why you might want to use json_object_object_get_ex() instead.

Issue #205: use _strdup instead of strdup on Windows.

Merge branch 'doctaweeks-fixes-for-upstream'

Fix a few places that needed adjustment for the size_t changes, including updating the range checks to use a calculated SIZE_T_MAX.

Merge branch 'fixes-for-upstream' of https://github.com/doctaweeks/json-c into doctaweeks-fixes-for-upstream

Merge pull request #193 from jplitza/master

Make serialization format of doubles configurable

Export json_object_double_to_json_string() and use custom format string

Fix code in docs for json_object_new_double_s()

Add tests for the json_object_array_del_idx() function.

Merge branch 'Protovision-master'

Merge branch 'master' of https://github.com/Protovision/json-c into Protovision-master

Merge pull request #216 from Jehan/master

configure: check realloc with AC_CHECK_FUNCS() to fix cross-compilation.

Merge pull request #218 from sevan/patch-1

Use a macro to indicate C99 to the compiler

Merge pull request #219 from rouault/low_heap_robustness_fixes

Fix various potential null ptr deref and int32 overflows

Merge pull request #220 from hschaa/master

Add utility function for comparing json_objects

Merge pull request #222 from chenha0/master

Fix issue #221: JSON_C_TO_STRING_NOSLASHESCAPE works incorrectly

Merge pull request #223 from minaguib/master

Clarify json_object_get_string documentation of NULL handling & return

Clarify json_object_get_string documentation of NULL handling & returning

Fix issue #221: JSON_C_TO_STRING_NOSLASHESCAPE works incorrectly

Tests added.

Add some basic tests for verifying json_object_equal behavior

Do some basic checks on ints, doubles, strings, arrays and "complex" objects.

Add utility function for comparing json_objects

Fix various potential null ptr deref and int32 overflows

This fix errors that can happen when ingesting very large JSON files
when hitting the maximum heap size of the process.

Use a macro to indicate C99 to the compiler

The [AC_PROG_CC_C99 macro](https://www.gnu.org/software/autoconf/manual/autoconf-2.64/html_node/C-Compiler.html) sets the complier to the C99 standard if the compiler does not default to such behaviour (albeit gnu99). This is better than specifically hardcoding flags

configure: check realloc with AC_CHECK_FUNCS() to fix cross-compilation.

AC_FUNC_REALLOC is messed up when cross-compiling, ending up in failed
build with "undefined reference to `rpl_realloc'" error.
AC_CHECK_FUNCS will work both with native and cross builds.

Add const qualifiers to several functions that don't modify the json_object.

Issue #137: remove config.h.in from change control.

Merge commit '2be921d88376e78f84d79aafa6db2714da804e59'

Merge pull request #209 from rgerhards/fix-regression

fix regression from 2d549662be832da838aa063da2efa78ee3b99668

Fix issue #201: add a JSON_C_TO_STRING_NOSLASHESCAPE flag to turn off escaping of forward slashes.

Merge pull request #198 from unmole/master

Fix possible memory leak and remove superfluous NULL checks before free()

Use size_t for json object array ops

Use size_t for array list length and size

fix regression from 2d549662be832da838aa063da2efa78ee3b99668

That commit introduced read-only keys, but when the hash table
was resized, that attribute was not preserved. This resulted in
an invalid free at time of table destruction.

Merge pull request #199 from Nzbuu/fix_vs_build

Fix build in Visual Studio

Merge pull request #200 from Nzbuu/ci_build

Add build scripts for CI platforms

Appveyor: Visual Studio v140 no longer fails

Appveyor: Create artefact with library and include files only

Use AX_APPEND_COMPILE_FLAGS() to check the various compile flags, such as -Wall, to ensure the compile supports.

Remove the AC_FUNC_MALLOC check, since we don't depend on the malloc(0) behavior it checks for, and we don't provide a rpl_malloc() implementation anyway.

Travis: add osx and clang builds

Travis: skip install step

Add Travis build script

Appveyor: Allow failing builds against newest toolset

Add initial version of  Appveyor build script

Fix build in Visual Studio

Remove superfluous NULL checks

Fix possible memory leak