summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* Fix one more assert("!invalid cint_type") to use json_abort(...) instead.Eric Haszlakiewicz2020-04-101-1/+1
|
* Issue #568: fix the strtoll and strtoull handing so config.h ends up ↵Eric Haszlakiewicz2020-04-093-31/+30
| | | | creating defines for those only when needed, which should exclude mingw environments.
* Remove autoconf files from .gitignore.Eric Haszlakiewicz2020-04-092-47/+13
|
* Update the oss-fuzz build script to try to get it to work with cmake.Eric Haszlakiewicz2020-04-091-7/+29
|
* Remove autoconf machinery, we're using cmake now. Update the release ↵Eric Haszlakiewicz2020-04-0911-853/+83
| | | | checklist to account for that, plus fill in other tasks and clean it up a bit.
* osx builds also need -DDISABLE_BSYMBOLIC=ONEric Haszlakiewicz2020-04-081-1/+5
|
* I have NFC why osx fails when "set -e" is in effect, but leave it off and ↵Eric Haszlakiewicz2020-04-081-8/+2
| | | | 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.Eric Haszlakiewicz2020-04-081-1/+1
|
* Try yet again to fix the osx build, this time do set +eEric Haszlakiewicz2020-04-081-0/+1
|
* Another attempt at debugging the travis osx build, this time just ignore errors.Eric Haszlakiewicz2020-04-081-3/+6
|
* Wait a while before exiting travis to allow output to flush.Eric Haszlakiewicz2020-04-081-0/+2
|
* Apparently cmake is already installed on travis osx builds. Try another ↵Eric Haszlakiewicz2020-04-081-4/+5
| | | | approach to figure out what was failing.
* Install cmake on osx travis builds. s/make check/make test/Eric Haszlakiewicz2020-04-081-1/+4
|
* Perform the travis build in a subdir, so the distcheck target works.Eric Haszlakiewicz2020-04-081-2/+2
|
* Switch travis to use cmake instead of autoconf.Eric Haszlakiewicz2020-04-081-7/+6
|
* Add includes and split off json_types.h to help ensure that headers can be ↵Eric Haszlakiewicz2020-04-068-60/+92
| | | | included in any order.
* Don't let clang-format rearrange Windows includes, since they need to be in ↵Eric Haszlakiewicz2020-04-061-1/+3
| | | | a particular order.
* Format random_seed.c with clang-format, with key asm sections excluded.Eric Haszlakiewicz2020-04-061-70/+79
|
* Update the style guide to mention that we're using clang-format now.Eric Haszlakiewicz2020-04-061-15/+16
|
* Merge pull request #555 from dota17/chang_format_3Eric Haszlakiewicz2020-04-0355-3084/+3328
|\ | | | | Format json-c with clang-format tool
| * clang-format the filesdota172020-04-0354-2781/+2928
| |
| * add the disabling formatting coments and adjust the partial code manulydota172020-04-0313-310/+354
| |
| * add optionsdota172020-04-031-2/+52
| |
| * clang-formatdota172020-04-031-0/+3
|/
* Merge pull request #563 from robybeen/masterEric Haszlakiewicz2020-04-022-2/+2
|\ | | | | Changed order of calloc args to match stdlib
| * Changed order of calloc args to match stdlib (2)Robert2020-04-021-1/+1
| | | | | | 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 stdlibRobert2020-04-021-1/+1
|/ | | 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_addEric Haszlakiewicz2020-04-029-1/+89
|\ | | | | Bugfix and testcases supplements
| * add test casesdota172020-03-319-1/+89
| |
* | Merge pull request #561 from dota17/add_badgeChen2020-03-311-0/+3
|\ \ | |/ |/| add the badge in README.md and test the coveralls
| * add the badge in README.md and test the coverallsdota172020-03-301-0/+3
|/
* Merge pull request #541 from dota17/coveralls_finalChen2020-03-281-0/+28
|\ | | | | add coveralls auto tool to json-c
| * modify the coveralls configuration filedota172020-03-232-8/+18
| |
| * add coveralls auto tool to json-cdota172020-02-182-0/+18
| |
* | Merge pull request #558 from dota17/doc_message_modifyChen2020-03-201-1/+1
|\ \ | | | | | | modify the doc target message
| * | modify the doc target messagedota172020-03-191-1/+1
|/ /
* | Merge pull request #556 from Jehan/wip/Jehan/fix-broken-mingw-w64Eric Haszlakiewicz2020-03-163-5/+29
|\ \ | | | | | | Fixes various Wreturn-type and Wimplicit-fallthrough errors on Mingw-w64
| * | Fixes various Wreturn-type and Wimplicit-fallthrough errors on Mingw-w64Jehan2020-03-163-5/+29
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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_unistallEric Haszlakiewicz2020-03-102-1/+12
|\ \ | | | | | | Add uninstall target in cmake
| * | delete -r in uninstalldota172020-03-111-1/+1
| | |
| * | Add uninstall target in cmakedota172020-02-272-1/+12
| | |
* | | Merge pull request #547 from dota17/assert_testEric Haszlakiewicz2020-03-034-5/+13
|\ \ \ | | | | | | | | modify json-c default build type, and fix up the assert() errors in t…
| * | | modify RELEASEdota172020-03-031-1/+1
| | | |
| * | | update testcase and delete debug -O0 builddota172020-03-023-9/+4
| | | |
| * | | modify json-c default build type, and fix up the assert() errors in testcasedota172020-02-294-5/+18
| |/ /
* | | Follow up the PR#542: improve assert() calls, simplify code in ↵Eric Haszlakiewicz2020-03-021-21/+18
| | | | | | | | | | | | json_object_equal().
* | | Merge pull request #542 from dota17/adduint64_finalEric Haszlakiewicz2020-03-0123-50/+526
|\ \ \ | |/ / |/| | add uint64 data to json-c
| * | update json_object.c and testcase, delete json_object_uint_inc()dota172020-02-284-88/+19
| | |
| * | modify partial functions and testcase, in order to support automatic ↵dota172020-02-287-42/+169
| | | | | | | | | | | | conversion for int64/uint64
| * | modify the json_object, replace c_int64/c_uint64 with struct{union{int64, ↵dota172020-02-277-87/+111
| | | | | | | | | | | | uint64},...}