Merge pull request #589 from Xyene/detect-broken-rdrand

Detect broken RDRAND during initialization

Detect broken RDRAND during initialization

Some CPUs advertise RDRAND in CPUID, but return 0xFFFFFFFF
unconditionally. To avoid locking up later, test RDRAND during
initialization, and if it returns 0xFFFFFFFF, mark it as nonexistent.

Fixes #588.

Merge pull request #590 from Xyene/fix-cpuid-segfault

Fix segmentation fault in CPUID check

Fix segmentation fault in CPUID check

Fix cmake-configure to accept "--prefix=<foo>" in addition to "--prefix <foo>" (see also Issue #591)

Add an initial version of a benchmarking harness for json-c, to be able to more easily compare the performance of different library versions.

Extend the CMakeLists.txt in the apps directory to be usable as a standalone build, to link against other versions of json-c.
Tweak json_parse.c slightly to allow it to build against older json-c versions.

The json_parse command line app doesn't build on Windows, disable it.

Add an apps directory, and a json_parse program to parse an input file and report on memory usage.
This is intended to provide a way, during development, to test out the memory
and performance impacts of a change.

Add a JSON_TOKENER_ALLOW_TRAILING_CHARS flag for json_tokener_set_flags() to allow multiple objects to be parsed from input even when JSON_TOKENER_STRICT is set.

Add an tok_flags field to explicitly specify tokener flags in test_parse and eliminate the previous bogus calls to json_tokener_set_flags()

Issue #471: always create directories with mode 0755, regardless of umask.

Issue #585: don't install config.h

Add a few missing git commands to the release checklist, and change the S3 storage to "Standard", since it's actually (barely) cheaper than "Reduced Redundancy" now.

Update the master branch to version 0.0.14.99

Fill in the ChangeLog for the upcoming 0.14 release.

Fill in a few more pre-release steps to match what we've done for the 0.14 release.
Move the list of contributors to the AUTHORS file.

Rewrite the issued_closed_for_* files so they display a bit nicer.

Explicitly mark several things in json_tokener deprecated.

Re-do clang-format.

Make json_abort() internal to json_object.c

Make json_tokener_validate_utf8() internal to json_tokener.c, and improve the docs a bit.

Merge pull request #582 from dota17/list_closed_issue_pr

add the list of issues& prs closed

update

manual adjustment

add the list of issues& prs closed

Merge pull request #524 from dota17/addTestCase_obj_token

Increase coverage

Merge pull request #581 from Jehan/wip/Jehan/no-strict-prototypes-windows

CMakeLists: do not enforce strict prototypes on Windows.

remove unsuitable case

CMakeLists: do not enforce strict prototypes on Windows.

On Windows, or at least when cross-built with Mingw-w64, build fails
because strict prototype fails on an included file (thus nothing we can
do about in json-c code):

> from /home/jehan/dev/src/json-c/json_util.c:44:
> /home/jehan/.local/share/crossroad/roads/w64/json-c/include/minwindef.h:196:3: error: function declaration isn't a prototype [-Werror=strict-prototypes]
>   196 |   typedef INT_PTR (WINAPI *FARPROC) ();
>       |   ^~~~~~~
> /home/jehan/.local/share/crossroad/roads/w64/json-c/include/minwindef.h:197:3: error: function declaration isn't a prototype [-Werror=strict-prototypes]
>   197 |   typedef INT_PTR (WINAPI *NEARPROC) ();
>       |   ^~~~~~~
> /home/jehan/.local/share/crossroad/roads/w64/json-c/include/minwindef.h:198:3: error: function declaration isn't a prototype [-Werror=strict-prototypes]
>   198 |   typedef INT_PTR (WINAPI *PROC) ();
>       |   ^~~~~~~

Let's just disable the errors for Windows build.

Merge pull request #580 from besser82/topic/besser82/fix_cmake_tests

Fix CMake tests for enforced strict prototypes.

Fix CMake tests for enforced strict prototypes.

Merge pull request #527 from dota17/arraylist_test

Arraylist testcase

Merge pull request #579 from besser82/topic/besser82/strict-prototypes

Enforce strict prototypes.

clang-format two test_util_file.c and test_object_iterator.c

Increased the test coverage of json_util.c from 76% to 90.3%.

Increased the test coverage of json_object_iterator.c from 0% to 100%

clang-format test1.c

fix valgrind errors

testcase for array_list

Enforce strict prototypes.

Merge pull request #578 from besser82/topic/besser82/pkgconfig_install

CMake: Install pkgconfig file in proper location by default

CMake: Install pkgconfig file in proper location by default

The default location for pkconfig files on most systems is:
  ${CMAKE_INSTALL_LIBDIR}/pkgconfig

Thus the file should get installed in there by default.

Merge pull request #577 from besser82/topic/besser82/json_c_version_test_verbose_on_fail

Be verbose on failing json_c_version test.

Be verbose on failing json_c_version test.

Merge pull request #576 from besser82/topic/besser82/increase_coverage

Test coverage for json_c_version.

Merge pull request #575 from besser82/topic/besser82/cosmetics

Small cosmetics.

Test coverage for json_c_version.

Remove multiple trailing newlines at EOF.

gitignore: Add build folder.

Clean trailing white-space.

Fix line terminators to be UNIX.

Update README.md to remove autoconf instructions in favor of cmake.

Merge pull request #572 from besser82/topic/besser82/cmake_fixes

Small fixes to CMakeLists

Merge pull request #573 from besser82/topic/besser82/coverage

Fix coveralls submission.

Fix coveralls submission.

coveralls-cpp is dead and does not work anymore.
Fortunately there is coveralls-lcov available.

CMake: Check whether BSYMBOLIC is supported by the linker.

Linking with '-Bsymbolic-functions' is supported for ELF-binaries, only.

CMake: Build Doxygen documentation out-of-tree.

CMake: Fix appending of linker flags.

There was a typo in `CMAKE_SHARED_LINKER_FLAGS`.

List the set of contributors since the 0.13 branch

Fix one more assert("!invalid cint_type") to use json_abort(...) instead.

Issue #568: fix the strtoll and strtoull handing so config.h ends up creating defines for those only when needed, which should exclude mingw environments.

Remove autoconf files from .gitignore.

Update the oss-fuzz build script to try to get it to work with cmake.

Remove autoconf machinery, we're using cmake now.  Update the release checklist to account for that, plus fill in other tasks and clean it up a bit.

osx builds also need -DDISABLE_BSYMBOLIC=ON

I have NFC why osx fails when "set -e" is in effect, but leave it off and clean up the rest of the debuging code in the travis config.

Not all systems (e.g. osx) have malloc.h, include stdlib.h instead.

Try yet again to fix the osx build, this time do set +e

Another attempt at debugging the travis osx build, this time just ignore errors.

Wait a while before exiting travis to allow output to flush.

Apparently cmake is already installed on travis osx builds.  Try another approach to figure out what was failing.

Install cmake on osx travis builds.  s/make check/make test/

Perform the travis build in a subdir, so the distcheck target works.

Switch travis to use cmake instead of autoconf.

Add includes and split off json_types.h to help ensure that headers can be included in any order.

Don't let clang-format rearrange Windows includes, since they need to be in a particular order.

Format random_seed.c with clang-format, with key asm sections excluded.

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…