Use the correct number of parens in JANSSON_VERSION_HEX macro

ANSI C requires struct initializers to be constant expressions

Closes GH-17.

Fix json_object_size() return value

Return 0 as documented if the argument is not a JSON object.

Closes GH-18.

Enhance portability of va_copy()

va_copy() is a C99 feature. In C89 implementations, it's sometimes
available as __va_copy(). If not, memcpy() should do the trick.

Fix a declaration after statement

While at it, add -Wdeclaration-after-statement to AM_CFLAGS to catch
these in the future.

Distribute doc/upgrading.rst

Closes GH-16.

Check documentation in make distcheck

It seems that the only way to do this is to use the dvi make target,
as it's built in make distcheck after running configure.

jansson 2.0

Fix packing of invalid UTF-8 strings

Update README

tests: Add missing json_decref() calls to suites/api/test_unpack.c

Remove invalid subdirs from test/suites/Makefile.am

doc: Change the title of the upgrading section

Small documentation tweaks

- Capitalize section titles correctly

- Rename "Building on Windows" to "Building on non-Unix systems" and
  tweak the text accordingly.

- Remove the "Installing Prebuilt Binaries" section. This info is
  better in the web instead, as it may change.

- Mention that Sphinx 1.0 or newer is required.

Add "Upgrading from older releases" to docs, tweak CHANGES a bit

Add a CHANGES entry for version 2.0

Wow, lots of stuff. And it's been a long while after the last release.

Add lots of tests for pack/unpack code, fix bugs found

Closes GH-12.

Move json_error_t documentation to its own section

json_error_t is used in the decoder and in the pack and unpack API's,
so it's better to have a separate section for it.

Truncate error source from start, not end, if it's too long to fit

It's more helpful to see "...bar/baz.json" instead of "/long/path/to".

Refactor decoder input stream

- Add a new field position to the json_error_t structure. This is the
  position in bytes from the beginning of the input.

- Keep track of line, column and input position in the stream level.
  Previously, only line was tracked, and it was in the lexer level, so
  this info was not available for UTF-8 decoding errors.

- While at it, refactor tests so that no separate "stripped" tests are
  required. json_process is now able to strip whitespace from its
  input, and the "valid" and "invalid" test suites now use this to
  test both non-stripped and stripped input.

Closes GH-9.

tests: Use "make STOP=1 check" to stop on first failure

Add custom memory allocation

Thanks to Basile Starynkevitch for the suggestion and initial patch.
Thanks to Jonathan Landis and Deron Meranda for showing how this can
be utilized for implementing secure memory operations.

Resolve __va_list_tag ** -> va_list * type errors with clang and future GCCs

Functions taking va_args are munged to receive arguments of type
'__va_list_tag *'. This patch uses va_copy to coerce them to the expected type
so we don't get compiler errors.

Tested on x86_64, both 32-bit and 64-bit compiles.

Reported-By: Basile Starynkevitch <basile@starynkevitch.net>

Fix an unpack example in the docs

Tweak the default validation behaviour of the unpack API

Now, by default, unpacking doesn't check that all array and object
items are accessed. The check can be enabled globally by using the
JSON_STRICT flag (formerly JSON_UNPACK_ONLY), or on a per-value basis
by using the new '!' format character. The '*' format character is
still available to disable the global check on a per-value basis.

Use the Sphinx default theme again for documentation

The "haiku" theme doesn't show chapter contents in sidebar.

Documentation for pack and unpack functions

Add the 'I' format for both pack and unpack

Use 'f' for real and 'F' for number (real or integer) in unpack

unpack: Unify and enhance error messages

Give the type name instead of its enum index in the error message when
an unexpected value is encountered while unpacking.

Expand the pack/unpack API, implement unpack flags

Expand the pack/unpack API: json_(un)pack is the simple version with
no error output, json_(un)pack_ex has error output and flags,
json_v(un)pack_ex is a va_list version of json_(un)pack_ex.

Implement unpacking flags:

- JSON_UNPACK_ONLY turns off extra validation, i.e. array and object
  unpacking doesn't check that all items are unpacked. This is really
  just convenience for not adding '*' after each object and array.

- JSON_VALIDATE_ONLY turns off unpacking, i.e. no varargs are expected
  and nothing is unpacked.

Add validation features to json_unpack()

* By default, json_unpack() now checks that all items of arrays and
  objects are unpacked. This is useful for validation.

* Add format specifier '*' to suppress this check for individual
  arrays and objects. '*' must appear as the last format specifier
  before the closing ']' or '}'.

Rename variadic.c to pack_unpack.c

Refactor json_unpack()

* Use the format string scanner from the previous commit (with the
  simpler separator skipping semantics)

* Split json_vnunpack() to three separate functions for unpacking
  objects, arrays and "simple" values

* Always return 0 on success, 1 on error

This shaves around 100 more lines from the original implementation.

Refactor json_pack()

* Implement a "scanner" that reads the format string, maintaining state

* Split json_vnpack() to three separate functions for packing objects,
  arrays and simple values. This makes it more clear what is being
  packed, and the object and array structures become more evident.

* Make the skipping of ignored character simpler, i.e. skip ':' and
  ',' independent of their context

This patch shaves around 80 lines of code from the original
implementation.

Unify style

Update copyright notices for 2011

Build cleanly with gcc's -Werror=declaration-after-statement flag

Remove debugging printf

Make json_pack/json_unpack() recursive

Note that we pass va_list pointers around instead of just va_lists, which
would seem more intuitive. This is necessary since the behaviour of va_lists
passed as function parameters is finicky. Quoth stdarg(3):

If ap is passed to a function that uses va_arg(ap,type) then the value
of ap is undefined after the return of that function.

The pointer-passing strategy is used by Python's Py_BuildValue() for the same
purpose.

Emphasize the constness of the return value of json_string_value()

Closes GH-7.

Document the version info constants

Make int32_t available when the configure script is not used

Add version info to jansson.h

Enhance error reporting

This patch adds two new fields to the json_error_t struct: column and
source. It also adds functions to populate json_error_t internally.

The column field is not currently used, but it will be utilized in the
decoder and pack/unpack functions.

Move and enhance documentation on json_error_t

Make json_error_t transparent again

After looking at the new code for a few days, I didn't like it
anymore. To prepare for the future, a few fields will be added to the
json_error_t struct later.

This reverts commit 23dd078c8dcb17fd29d3b69f082b5b93f5f13b8f. Some
adjustments were needed because of newer commits.

Adds json_pack / json_unpack variadic functions.

Remove all "Added in version 1.x" info from documentation

As 2.0 will be backwards incompatible anyway, there's no need to
record which features were new in which 1.x releases.

Make json_error_t opaque

All decoding functions now accept a json_error_t** parameter and set
it to point to a heap-allocated json_error_t structure if an error
occurs. The contents of json_error_t are no longer exposed directly, a
few functions to do it have been added instead. If an error occurs,
the user must free the json_error_t value.

This makes it possible to enhance the error reporting facilities in
the future without breaking ABI compatibility with older versions.

This is a backwards incompatible change.

Merge branch '1.3'

Run Sphinx without the -W flag when building documentation

In "make html", don't use the -W flag with Sphinx. This makes it
possible to create the documentation with Sphinx 1.0 without errors,
as the warning about using old-style C markup isn't turned to an
error.

Don't build documentation in "make check". Instead, add a new make
target "check-doc" to build the documentation with the -W flag.

Use Sphinx 1.0, change the HTML theme

Add a section describing how to build on Windows

On Windows, typedef int to int32_t

There's no inttypes.h or stdint.h in the stdlib on Windows.

Add jansson_config.h.win32

As the configure script cannot be run on Windows, give the users a
jansson_config.h that they can use directly.

Move max() to jansson_private.h, define only if not already defined

On some platforms (Visual C++ for one) the standard library already
defines max() as a macro.

Don't use designated struct initializers

It's a C99 feature and not available on all compilers (Visual C++, for
example).

Use config.h only if it exists

This makes it easier to compile without the configure script (e.g. on
Windows).

Replace all occurences of inline with JSON_INLINE

This makes it easier to compile without the configure script (e.g. on
Windows).

Merge branch '1.3'

Conflicts:
doc/apiref.rst
src/jansson_private.h

Set the version number to 2.0pre

Clarify and document the integer type configuration

Add a flags parameter to all decoding functions for future needs

As of now, the parameter is unused, but may be needed in the future.
I'm adding it now so that in the future both API and ABI remain
backwards compatible as long as possible.

This is a backwards incompatible change.

Change the maximum indentation size to 32 spaces in encoder

This is to free up bits from the flags parameter of json_dump
functions. I'm pretty sure no-one needs 256 spaces of indentation when
pretty-printing JSON values...

This is a backwards incompatible change.

Change the underlying type of JSON integer from long to json_int_t

json_int_t is typedef'd to long long if it's supported, or long
otherwise. There's also some supporting things, like the
JSON_INTEGER_FORMAT macro that expands to the printf() conversion
specifier that corresponds to json_int_t's actual type.

This is a backwards incompatible change.

Make object_key_t portable

A flexible array member is unportable. Use a table of length 1
instead. This needs some adjustment to the memory allocatio, too.

Beautify the container_of macro

Use offsetof instead of zero pointer dereference trickery.

Replace inline with JSON_INLINE in json_object_iter_set() declaration

Check for gcc before setting gcc specific CFLAGS

Move site configuration to jansson_config.h

This way, more site configuration can be more easily added later.

Change JSON integer's underlying type from int to long

This is a backwards incompatible change.

Clarify the documentation

Couple some string and number information from the RFC conformance
chapter in the API reference, and refer to the RFC conformance chapter
from API reference for more information.

Also, state more clearly that a JSON text must have an array or object
as the top-level value, and better document the string comparison
performed by json_equal().

Unify unsigned integer usage in the API

Replace all occurences of unsigned int and unsigned long with size_t.

This is a backwards incompatible change, as the signature of many API
functions changes.

configure.ac: Remove unneeded AC_PROG_CXX

Add a few missing changes to CHANGES for v1.3

These were forgotten when releasing.

jansson 1.3

Remove the C++ interface

Merge branch '1.2'

Fix a few memory leaks in tests

No changes to the actual library code.

Merge branch '1.2'

Clarify the documentation on reference stealing

Provide an example usage pattern for reference stealing functions.
This way the user (hopely) sees more clearly how the reference
stealing functions are meant to be used.

Zero the visited flag after an encoding error

When encoding an array or object ends in an error, the visited flag
wasn't zeroed, causing subsequent encoding attempts to fail. This
patch fixes the problem by always zeroing the visited flag.

Zero the visited flag after encoding an empty array or object

Encoding an empty array or object worked, but encoding it again
(possibly after adding some items) failed, because the visited flag
(used for detecting circular references) wasn't zeroed.

Merge branch '1.2'

load.c: Make stream_init() static

Merge branch '1.2'

Conflicts:
CHANGES
configure.ac
doc/conf.py

jansson 1.2.1

doc: Distribute conformance.rst

Merge branch '1.2'

Conflicts:
configure.ac

Require autoconf 2.60

The AC_TYPE_INT32_T macro first appeared in autoconf 2.60.

Support compilers that don't have the "inline" keyword

Use AC_C_INLINE autoconf macro, include config.h where needed, and add
a define of JSON_INLINE to jansson.h that has the correct "inline"
keyword.

doc: Add chapter on RFC conformance

Thanks to Deron Meranda for providing the initial text.

Estimate real number underflows with 0.0

Earlier it was a decoding error.

Enhance tests for null byte

Merge branch '1.2'

Fix reference counting on true, false and null

Initialize their reference counts to (unsigned int)-1 to disable
reference counting on them. It already was meant to work like this,
but the reference counts were just initialized to 1 instead of -1.

Thanks to Andrew Thompson for reporting this issue.

Ignore temporary files *~

c++ wrapper: add missing 'inline' statements to various constructors

the missing 'inline' leads to duplicated symbols if the header is
included into two separately compiled files.

Don't include stdint.h anywhere

This should have fixed by commit 28682322, but there was one #include
left in utf.c. It now includes utf.h instead of stdint.h.