Merge pull request #299 from qlyoung/perf-improvements

Improve json_object -> string performance

Restore sprintbuf(), add macro for string literals

Hawciz pointed out that the previous commit modifies the public
interface of printbuf. Per his suggestion, sprintbuf() was restored
and a new pair of macros was added that wraps printbuf_memappend().

Using a wrapper macro instead of modifying sprintbuf() also reduces
function call overhead, bringing total performance gains to
approximately 400%.

Use strdup instead of strndup in test1.c, there's no need for the latter because json_object_to_json_string_length() will always return a properly sized string.

Improve json_object -> string performance

Removes variadic prints for ~3x performance improvement.

Issue #295: also check if size_t is the size of long long, to help support 64-bit Windows platforms.

Ignore tests/test_float

Change a memcpy that should be a memmove within json_pointer_get, and fix memory leaks in of one the json_pointer tests.

Note some minimum versions needed for autoconf, etc...

Issue#292: bump the required version of configure listed in configure.ac so a more useful error message is generated, rather than failing later because the extra macros from ./autoconf-archive/m4/ didn't get loaded.

Merge pull request #289 from jgerhards/i-278

bugfix: floating point representaion without fractional part

testbench: add test for floating point representation

bugfix: floating point representaion without fractional part

closes https://github.com/json-c/json-c/issues/278

Merge pull request #287 from commodo/json_pointer_va_args

json_pointer: extend setter & getter with printf() style arguments

vasprintf_compat.h: spin-off this compat header ; use math_compat.h as template

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

strdup_compat.h: re-spin this compat header ; use math_compat.h as template

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

test_json_pointer: update test with a few printf variants

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

json_pointer: add json_pointer_getf/setf() function variants

These include support for printf() style args for path.

Adds support for calling with 'json_pointer_getf(obj, &res, "/foo/%d/%s", 0, bar)'
style args.

Makes it easier for doing more dynamic stuff/magic, without
needing to use vasprintf() externally.

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

json_pointer.c: fix whitespace

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

Merge pull request #285 from ebassi/revert-strdup-compat

Revert "compat/strdup.h: move common compat check for strdup() to own…

Revert "compat/strdup.h: move common compat check for strdup() to own file"

This reverts commit aaba8c1080533133329086e47f4f45981dfdd1ee.

This commit breaks builddir != srcdir build, but, more importantly, it
also adds a dependency on a header, "config.h", which is not installed
and it's supposed to be private — since it's generated at configuration
time and it's not meant to be used by projects compiling against a
library.

Merge pull request #283 from commodo/json_pointer

json_pointer: add first revision based on RFC 6901

tests: add test_json_pointer test

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

tests/strerror_override.c: fix compilation error

I got this on Mac OS X at least.
Not sure if it shows up in other envs

error:
```
strerror_override.c:53:13: error: incompatible redeclaration of library function 'strerror' [-Werror,-Wincompatible-library-redeclaration]
const char *strerror(int errno_in)
```

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

json_pointer: add first revision

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

compat/strdup.h: move common compat check for strdup() to own file

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

Handle NULL objects in json_object_get_userdata() by returning NULL, but abort in json_object_set_userdata() since we can't actually do anything with the userdata.

Issue #280: add a few assert() calls to ensure objects are of the appropriate type.

Use json_visit() to clear the serializer data set when parsing so the output from test_parse reflects the actual values parsed.

Add a json_c_visit() function to provide a way to iterate over a tree of json-c objects.

Fix broken key-order test in test_compare.

Merge pull request #272 from sgerbino/master

Allows for compilation with CMake on macOS

Merge pull request #277 from ALLTERCO/json_object_set_xxx

Json object set xxx

try restore windows automated builds

fix compiler warning for int sizes

string set and tests

gitignore test; add double value set checks in test

some simple tests

Issue #275: fix out of bounds read when handling unicode surrogate pairs.

some basic set

Merge pull request #273 from konoal/master

fixed wrong object name in json_object_all_values_equal

Merge pull request #274 from pepevel/master

Support for 64 bit pointers on Windows

Support for 64 bit pointers on Windows

fixed wrong object name in json_object_all_values_equal

Changed CMake variables and adding output for debugging

Target C99 standards to get rid of warnings

Modified the execute_process commands

Changed PROJECT_SOURCE_DIR to CMAKE_CURRENT_SOURCE_DIR to fix building when used as external project

Modified CMakeLists.txt to use autotools to generate configuration for Unix systems

Lowered CMake required version so that Travis CI is able to build, moved include folder to include/json-c to match the behavior of autotools

Trailing slash seems necessary in the install command to get contents

Forget DESTINATION in CMake install command

We need to also install the generated configuration files

Fixed typo in CMakeList.txt FILES_MATCHING

Added install instructions for CMake

Resolves json-c/json-c#271 by adding MacOS configuration files and modifies CMakeLists.txt to use them

json_object_set_boolean for upstream style check

Merge pull request #267 from sixlettervariables/patch-1

Removes me as Win32 maintainer, because I'm not.

Removes me as a maintainer, because I'm not.

Updates README-WIN32.html to remove my status as a maintainer (I'm not). Also removes ancient porting notes and adds a reference to json_config.h.win32 which is where most folks should go to start with Windows issues.

Issue #263: add const so the prototype for json_object_object_add_ex() matches the definition.

Add Cmakefile

Fix memory leak and inconsistent output (due to varying fd numbers) in test_util_file.

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.