rurban / Cpanel-JSON-XS

61%

master: 69%

LAST BUILD ON BRANCH dupkeys

branch: dupkeys

CHANGE BRANCH

x

Reset

• dupkeys
• 56testfixes
• JSON-PP-Boolean-gh40
• appveyor
• bom
• bool+stringify-gh41
• bool-gh53
• convert_blessed-gh37
• dagolden/preserve-nv-as-float
• decodebig-gh50
• free
• gccver
• gh105-blessed
• gh128-amgrec
• gh154-normfloat
• gh167-canon-tiedhash
• gh174-relaxed
• gh179-threads
• gh66-class-error
• gh74-nonchar
• gh82-decode_prefix
• gh84-need
• gh92-coverage
• inf-aix
• madsen-json_test_failure
• master
• merge-3.02
• mingw-nan-gh57
• mod-hvalue-gh45
• mojo-bool
• n_number_then_00
• overflows
• pod-spelling
• pp-options-gh47
• pr134
• pr138-rethrow
• pr157-meta
• pr42-SvRMAGICAL
• redefine-gh60
• rel-3.0204
• relcand
• s390x-ld
• shared
• smoke
• smoke-3.0214
• smoke-win
• smoke/4.13
• smoke/pr147
• squote-gh54
• string_not_in_unicode
• stringify-gh49
• stringify_blessed-gh37
• threads-shared
• travis
• unicode-nonchars
• unknown-gh46
• win32
• yes-no-gh39
• zero-mojibake

error on duplicate keys

contrary to the spectests, which classifies duplicate keys as allowable.
This is arguably a bug. see http://seriot.ch/parsing_json.php#24
RFC 7159 section 4 says that "The names within an object should be unique."
So it's either i_ (undefined) or n_ (errors).

Here we error by default and allow duplicate keys in relaxed mode.
See https://github.com/rurban/Cpanel-JSON-XS/pull/75
and https://github.com/makamaka/JSON-PP/pull/30