Update the style guide to mention that we're using clang-format now.

Merge pull request #555 from dota17/chang_format_3

Format json-c with clang-format tool

clang-format the files

add the disabling formatting coments and adjust the partial code manuly

add options

clang-format

Merge pull request #563 from robybeen/master

Changed order of calloc args to match stdlib

Changed order of calloc args to match stdlib (2)

Although it is currently working, it's worth to stick with the stdlib definition to avoid further problems

Changed order of calloc args to match stdlib

Although it is currently working, it's worth to stick with the stdlib definition to avoid further problems

Merge pull request #562 from dota17/testcase_add

Bugfix and testcases supplements

add test cases

Merge pull request #561 from dota17/add_badge

add the badge in README.md and test the coveralls

add the badge in README.md and test the coveralls

Merge pull request #541 from dota17/coveralls_final

add coveralls auto tool to json-c

modify the coveralls configuration file

Merge pull request #558 from dota17/doc_message_modify

modify the doc target message

modify the doc target message

Merge pull request #556 from Jehan/wip/Jehan/fix-broken-mingw-w64

Fixes various Wreturn-type and Wimplicit-fallthrough errors on Mingw-w64

Fixes various Wreturn-type and Wimplicit-fallthrough errors on Mingw-w64

This is a recent regression since commit
6359b798479d379a3202e02c6a938d9b40c0d856 which added various assert(0)
calls (often replacing return-s).
With Ming-W64 compiler, json-c build was failing with various errors of
the sort:

> /home/jehan/dev/src/json-c/json_object.c: In function 'json_object_int_inc':
> /home/jehan/dev/src/json-c/json_object.c:841:1: error: control reaches end of non-void function [-Werror=return-type]
>   841 | }
>       | ^
> In file included from /home/jehan/dev/src/json-c/json_object.c:17:
> /home/jehan/dev/src/json-c/json_object.c: In function 'json_object_get_double':
> /home/jehan/.local/share/crossroad/roads/w64/json-c/include/assert.h:76:4: error: this statement may fall through [-Werror=implicit-fallthrough=]
>    76 |   (_assert(#_Expression,__FILE__,__LINE__),0))
>       |    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> /home/jehan/dev/src/json-c/json_object.c:1070:7: note: in expansion of macro 'assert'
>  1070 |       assert(0);
>       |       ^~~~~~
> /home/jehan/dev/src/json-c/json_object.c:1072:3: note: here
>  1072 |   case json_type_boolean:
>       |   ^~~~

The problem is that Mingw-w64 does not consider assert() as a noreturn
(even assert(0)), because it has to be compatible by Microsoft
libraries. See the discussion here:
https://sourceforge.net/p/mingw-w64/bugs/306/

Instead let's create a new json_abort() function which is basically just
an abort() function with an optional message, for such cases where
abortion was non-conditional (using assert() and using the assertion
condition as a message here was clearly a misuse of the function). And
mark json_abort() as 'noreturn', as well as 'cold' for optimization
purpose (this is code we expect to never run, unless there is a bug,
that is).

Finally let's use this json_abort() instead of previous misused assert()
calls.

Merge pull request #546 from dota17/make_unistall

Add uninstall target in cmake

delete -r in uninstall

Merge pull request #547 from dota17/assert_test

modify json-c default build type, and fix up the assert() errors in t…

modify RELEASE

update testcase and delete debug -O0 build

Follow up the PR#542: improve assert() calls, simplify code in json_object_equal().

Merge pull request #542 from dota17/adduint64_final

add uint64 data to json-c

modify json-c default build type, and fix up the assert() errors in testcase

update json_object.c and testcase, delete json_object_uint_inc()

modify partial functions and testcase, in order to support automatic conversion for int64/uint64

Add uninstall target in cmake

modify the json_object, replace c_int64/c_uint64 with struct{union{int64, uint64},...}

Merge pull request #545 from dota17/make_doc

add doc target in cmake

add doc target on cmake

Merge pull request #544 from dota17/distcheck

Increase distcheck target in cmake

Merge pull request #543 from dota17/readme

Readme

Increase distcheck target in cmake

Update README.md

Added documentation for camke-configure

add uint64 data to json-c

add coveralls auto tool to json-c

Issue #539: use a internal-only serializer function in json_object_new_double_s() to avoid potential conflicts with user code that uses the json_object_userdata_to_json_string serializer.  Also, document the serializer-resetting behavior of json_object_set_double().

Issue #539: be sure to clean up at the end of test_set_value.

Issue #539: reset the serializer when json_object_set_double() is called and the current serializer is the one that json_object_new_double_s() used.

Merge pull request #531 from dota17/utf8test

validate utf-8 string

Merge pull request #536 from dota17/new_null

add json_object_new_null()

update comment

add json_object_new_null

update code

Merge pull request #533 from sunpoet/master

Fix "make check"

Fix make check

cc -DHAVE_CONFIG_H -I. -I..  -I.. -I../tests    -O2 -pipe  -fstack-protector-strong -fno-strict-aliasing  -Wall -Werror -Wcast-qual -Wno-error=deprecated-declarations -Wextra -Wno-unused-parameter -D_GNU_SOURCE -D_REENTRANT -MT test_parse.o -MD -MP -MF .deps/test_parse.Tpo -c -o test_parse.o test_parse.c
test_parse.c:256:14: error: adding 'int' to a string does not append to the string [-Werror,-Wstring-plus-int]
        { "null123" + 4,       4, 3, json_tokener_success, 1 },
          ~~~~~~~~~~^~~
test_parse.c:256:14: note: use array indexing to silence this warning
        { "null123" + 4,       4, 3, json_tokener_success, 1 },
                    ^
          &         [  ]
test_parse.c:258:12: error: adding 'int' to a string does not append to the string [-Werror,-Wstring-plus-int]
        { "nullx" + 4,         2, 0, json_tokener_error_parse_unexpected, 1 },
          ~~~~~~~~^~~
test_parse.c:258:12: note: use array indexing to silence this warning
        { "nullx" + 4,         2, 0, json_tokener_error_parse_unexpected, 1 },
                  ^
          &       [  ]
test_parse.c:260:25: error: adding 'int' to a string does not append to the string [-Werror,-Wstring-plus-int]
        { "{\"a\":1}{\"b\":2}" + 7,
          ~~~~~~~~~~~~~~~~~~~~~^~~
test_parse.c:260:25: note: use array indexing to silence this warning
        { "{\"a\":1}{\"b\":2}" + 7,
                               ^
          &                    [  ]
3 errors generated.
*** Error code 1

Stop.

test utf8

Merge pull request #526 from dota17/addTestCase_printbuf

Increased the test coverage of printbuf.c 82% to 92%.

Merge pull request #525 from dota17/addPointerTestcase

update pointer test case

Increased the test coverage of printbuf.c 82% to 92%.

Merge pull request #523 from dota17/updatetestcase

update tsetcase for tokener_c

Merge pull request #522 from dota17/addVisitTestcase

update json_visit testcase

update tsetcase for tokener_c

update json_visit testcase

update pointer test case

Merge pull request #519 from dota17/addTestCase_obj_token

Add test case obj token

pointer types discards qualifiers

add testcases of object and token

Merge pull request #512 from JaapKeuter/cmake_test

Properly append to CMAKE_C_FLAGS string

Properly append to CMAKE_C_FLAGS string

Contrary to other CMAKE variables the CMAKE_C_FLAGS variable is the
composed string of flags for the C compiler. It is therefore not a list
to append to. Current implementation results in these incorrect CFLAGS,
e.g., "-O2 -g -fblahblah;-UNDEBUG". Extending the CFLAGS this way
results in the proper CFLAGS, e.g., "-O2 -g -fblahblah -UNDEBUG".

Set cmake policy CMP0075, to have check_include_file use the value of CMAKE_REQUIRED_LIBRARIES (currently, adds -lm).  See issue #510.

Undefine NDEBUG for tests - cmake version.  See issue #501.

Merge pull request #501 from andy5995/iss_406-2

undefine NDEBUG for tests

Add a shim script to ease shift from autoconf to cmake.

Add a few missing features to the cmake setup that are present in configure.ac:
 Include all compiler warnings, and provide DISABLE_WERROR to make them not be errors.
 Define _REENTRANT, if setting it works.
 Set -Bsymbolic-functions, and provide DISABLE_BSYMBOLIC to turn that off.
 Implement the check for HAS_GNU_WARNING_LONG

Set the soversion in the cmake built library, and arrange for the file name to be generated appropriately too.

Fix test_util_file for VS2013 too, but skip all the tests for anything older than that because the limitations are too inconvenient.

Fix up the test_util_file test for builds on Windows VS2015.

Fix some Windows compile issues, add JSON_EXPORT's, fix bogus character escapes, define __func__ and omit unistd.h if needed.

Get the cmake build a bit closer to the autoconf one: include json_visit.h, and fix the version stamped in json-c.pc.

Build and run the tests as part of the cmake build.

Fix memory leaks in test_double_serializer, and make sure all tests return 0 at the end of main().

Add a quick way (JSONC_TEST_TRACE=1) to turn on shell tracing in tests.

Extend test_double_serializer to check NaN and Infinity handling.

Define vars earlier to fix old Windows builds.

Add a json_object_from_fd_ex() function, to allow the max nesting depth to be specified.

Add a test for serializing the double value -1.0.

Merge pull request #505 from grdowns/vcpkg-instructions

Add vcpkg installation instructions

Update dependencies

Add vcpkg installation instructions

Issue #488: use JSON_EXPORT on functions so they are properly exported on Windows.

Issue #463: fix newlocale() call to use LC_NUMERIC_MASK instead of LC_NUMERIC, and remove incorrect comment.
The second call to newlocale() with LC_TIME accidentally made things
 work because LC_TIME == LC_NUMERIC_MASK on some platforms.

Add a json_tokener_get_parse_end() function to replace direct access of tok->char_offset.

Minor cleanup of includes in a couple of tests.

Add an explicit cast to double to squash a -Wimplicit-int-float-conversion warning.
Though we will no longer be comparing exactly against INT64_MAX, this is ok
because any value of that magnitude stored in a double will *also* have been
rounded up, so the comparison will work appropriately.

undefine NDEBUG for tests

(closes #406)

Merge pull request #499 from andy5995/travis_valgrind

.travis.yml:test on more recent clang and gcc versions

add xenial default clang with CHECK enabled

add missing dist

manually use apt-get to install packages on bionic

add tests on bionic beaver

revert toolchain back to "test"

change key

use "non-test" ppa

Maybe something weird about how gcc is configured with the test
toolchain?

remove useless condition that shows logs

install doxygen so 'make distcheck' can succeed