Use jsonsl_path_match instead of jpr_match

The new function allows for smaller, simpler code in match.cc

I've also started moving away from the `is_arridx` variable in
jpr_compo

Use jsonsl_path_match instead of jpr_match

The new function allows for smaller, simpler code in match.cc

I've also started moving away from the `is_arridx` variable in
jpr_component_st, as this is implied by a `ptype` of JSONSL_PATH_NUMERIC
using the newer path_match semantics.

Change-Id: Iaae694d52d7caec44676007dd79a3d343c64557e
Reviewed-on: http://review.couchbase.org/57788
Well-Formed: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: buildbot <build@couchbase.com>

show more ...

MB-16695: Use mandatory JSON path escaping

Without path escaping we risk having invalid JSON. Additionally, some
sequences like '"' and '\"' might seem ambiguous (do they mean a literal

MB-16695: Use mandatory JSON path escaping

Without path escaping we risk having invalid JSON. Additionally, some
sequences like '"' and '\"' might seem ambiguous (do they mean a literal
escape-quote or just intend to place the quote). By restricting every
path component to be valid JSON we solve for inadvertently inserting bad
paths up the stack.

This commit also disables support for using \u-XXXX sequences in paths.
Despite them being valid JSON, they need to be normalized before they
are used in the parser. Support for u-escapes in paths will be present
when these escapes are properly normalized into utf8 within the path
parser.

Change-Id: Ia3c3bccf3b6e3006268635656d1c66ab4225a24f
Reviewed-on: http://review.couchbase.org/56617
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: buildbot <build@couchbase.com>

show more ...

C++ify component structures

Change-Id: I37de4782daa352a503e8f90afdb830d21518c382
Reviewed-on: http://review.couchbase.org/51036
Reviewed-by: Dave Rigby <daver@couchbase.com>
Test

C++ify component structures

Change-Id: I37de4782daa352a503e8f90afdb830d21518c382
Reviewed-on: http://review.couchbase.org/51036
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: buildbot <build@couchbase.com>

show more ...

Validate depth of new value

This checks to see if the new value is not too deep to be inserted. If
it's too big, an Error::VALUE_ETOODEEP error is returned.

Change-Id: I28bb1a9d

Validate depth of new value

This checks to see if the new value is not too deep to be inserted. If
it's too big, an Error::VALUE_ETOODEEP error is returned.

Change-Id: I28bb1a9d1f34684cbb013feb988e2e5f7e430301
Reviewed-on: http://review.couchbase.org/50915
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Mark Nunberg <mark.nunberg@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

Add LICENSE and copyright headers

Change-Id: Ibc9fcf211a2ad6ba68918bc0f6a2d494d34d8b47
Reviewed-on: http://review.couchbase.org/50796
Tested-by: buildbot <build@couchbase.com>
Re

Add LICENSE and copyright headers

Change-Id: Ibc9fcf211a2ad6ba68918bc0f6a2d494d34d8b47
Reviewed-on: http://review.couchbase.org/50796
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

Use long rather than ssize_t

Change-Id: Ie0649abe168478a6e1f70a164e3d8ae306c414b1
Reviewed-on: http://review.couchbase.org/50794
Tested-by: buildbot <build@couchbase.com>
Reviewe

Use long rather than ssize_t

Change-Id: Ie0649abe168478a6e1f70a164e3d8ae306c414b1
Reviewed-on: http://review.couchbase.org/50794
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

Remove C-style wrappers (1/2)

This adds some C++ methods and refactors the tests to use these new
methods rather than the old C-style wrappers.

Change-Id: Ifcc4042e6fd59ec3ec9b7

Remove C-style wrappers (1/2)

This adds some C++ methods and refactors the tests to use these new
methods rather than the old C-style wrappers.

Change-Id: Ifcc4042e6fd59ec3ec9b7c4a0ea176879c9efea0
Reviewed-on: http://review.couchbase.org/50602
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

Convert code to C++-only

This removes the 'extern "C"' declarations and fully converts the
headers into C++ components.

Existing C++ code using the "C" API will continue to func

Convert code to C++-only

This removes the 'extern "C"' declarations and fully converts the
headers into C++ components.

Existing C++ code using the "C" API will continue to function as opaque
function symbols are now simply macros to call into their C++
equivalents. Macros will be removed once dependent code is converted.

Change-Id: Ic2b5d05ecb22a5717600661bfcccdb3d84dee4dd
Reviewed-on: http://review.couchbase.org/50522
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

Fix build break on win32

* There is no ssize_t in Visual Studio 2013,
but size_t is actually signed

* Tons of link errors trying to use gtest, so its
disabled until we f

Fix build break on win32

* There is no ssize_t in Visual Studio 2013,
but size_t is actually signed

* Tons of link errors trying to use gtest, so its
disabled until we figure it out

Change-Id: If40ae9d06a802502d562aeae7490a49c317c2a7d
Reviewed-on: http://review.couchbase.org/50115
Tested-by: Trond Norbye <trond.norbye@gmail.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

Fix compiler warnings (from GCC 4.9)

Change-Id: Ia697521ac3fd673fffdd6f14919278f91cf07085
Reviewed-on: http://review.couchbase.org/50056
Tested-by: buildbot <build@couchbase.com>

Fix compiler warnings (from GCC 4.9)

Change-Id: Ia697521ac3fd673fffdd6f14919278f91cf07085
Reviewed-on: http://review.couchbase.org/50056
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Trond Norbye <trond.norbye@gmail.com>
Reviewed-by: Mark Nunberg <mark.nunberg@couchbase.com>

show more ...

Allow access to documents of depth 32

Use correct type (subdoc_ERRORS) for status codes

Instead of a generic uint16_t, use the correct type (subdoc_ERRORS)
for subjson status codes.

Allow "Last element" ([-1]) notation.

This will simplify the commandset of POP, and allow us to remove them
altogether.

Move src => subdoc

Self-contained subdoc src