History log of /5.5.2/platform/tests/cjson/ (Results 1 - 12 of 12)
Revision (<<< Hide revision tags) (Show revision tags >>>)Date Author Comments
(<<< Hide modified files)
(Show modified files >>>)
Revision tags: v6.0.3, v5.5.4, v5.5.0, v4.6.5
6696145f16-Feb-2018 Dave Rigby <daver@couchbase.com>

Replace INCLUDE_DIRECTORIES with TARGET_INCLUDE_DIRECTORIES

It is preferred to use target_include_directories() instead of
include_directories() to ensure that header dependancies are ex

Replace INCLUDE_DIRECTORIES with TARGET_INCLUDE_DIRECTORIES

It is preferred to use target_include_directories() instead of
include_directories() to ensure that header dependancies are explicit
(and we don't accidently depend on headers). See 'Effective Modern
CMake[1]' for background.

Replace include_directories() with target_include_directories() where
necessary, or simply remove include_directories() where it is
unnecessary - for example where a target already depends on a library
and the library has already declared its public includes.

To assist with this, an explicit 'platform_headers' interface library
has been created, targets which only wish to use platform's headers
(for example tests) can link to that library to get the correct search
paths.

[1]: https://gist.github.com/mbinna/c61dbb39bca0e4fb7d1f73b0d66a4fd1

Change-Id: I4483fc3d5c888ef7bf1ece2e875df5c90655d1a6
Reviewed-on: http://review.couchbase.org/90742
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Trond Norbye <trond.norbye@gmail.com>

show more ...

da5130e626-Feb-2018 Trond Norbye <trond.norbye@gmail.com>

Refactor: Split cJSON_Number into cJSON_Double

We barely use the double part in our code, so it's easier to
jet cJSON_Number mean a signed integer, and cJSON_Double
represent floatin

Refactor: Split cJSON_Number into cJSON_Double

We barely use the double part in our code, so it's easier to
jet cJSON_Number mean a signed integer, and cJSON_Double
represent floating points.

This means that we can use signed 64 integers without caring
about information loss (except for signness issues, but that
require deeper changes to cJSON)

Change-Id: Ib402e2e1fa5640199b518ed700b592b2be576969
Reviewed-on: http://review.couchbase.org/90056
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>

show more ...

bac3aa4e21-Feb-2018 Tim Bradgate <tim.bradgate@couchbase.com>

MB-27661 [2/n]: Add further API methods for cJSON

This adds four new API methods to help adding numbers to the JSON
object.

- cJSON_AddIntegerToObject
This method ad

MB-27661 [2/n]: Add further API methods for cJSON

This adds four new API methods to help adding numbers to the JSON
object.

- cJSON_AddIntegerToObject
This method adds a uint32_t to the cJSON object by casting it
to a double. This is safe as the IEEE-754 spec has 52 bits for
mantissa.

- cJSON_AddInteger64ToObject
This method adds a uint64_t to the cJSON object by casting it
to a double. This method performs a narrowing conversion so it
should be noted there is the potential for malformed data.

- cJSON_AddStringifiedIntegerToObject
This method adds an uint64_t to the cJSON object through the use
of a string representation of the number.

- cJSON_AddStringifiedSignedIntegerToObject
This method adds an int64_t to the cJSON object through the use
of a string representation of the number.

This also adds associated unit tests for these methods.

Change-Id: Ic39b7ce82ca0fbcd550aab8d076fce5e8a213094
Reviewed-on: http://review.couchbase.org/89812
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

1460328903-Oct-2017 Eugen-Alexandru Virtan <eugen.virtan@couchbase.com>

MB-26047:[2] Migrate away from using gethrtime()

Replace uses of gethrtime() with ProcessClock::now() and refactor
related code to use typesafe <chrono> time units.

Change-Id: I

MB-26047:[2] Migrate away from using gethrtime()

Replace uses of gethrtime() with ProcessClock::now() and refactor
related code to use typesafe <chrono> time units.

Change-Id: I6faf5fdcf49d1687176b279b1986b5a90949b5c3
Reviewed-on: http://review.couchbase.org/83969
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...

Revision tags: v5.0.0, v4.6.0, v4.6.0-DP, v4.5.1-MP1, v4.5.1
c9ca84bc18-Aug-2016 Dave Rigby <daver@couchbase.com>

MB-20769: Use cbmalloc for cJSON

cJSON performs memory allocation internally, and also allocates memory
which is returned (either directly or indirectly) to clients. For both
cases i

MB-20769: Use cbmalloc for cJSON

cJSON performs memory allocation internally, and also allocates memory
which is returned (either directly or indirectly) to clients. For both
cases it uses a set of malloc functions to perform allocation, which
default to the standard system malloc/free etc.

This patch changes the default malloc functions to be cbmalloc (the
user can still use the hook API to overide to something different). It
also fixes a couple of instances where the system functions were
incorrectly hard-coded.

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

show more ...

8616395e20-Sep-2016 Dave Rigby <daver@couchbase.com>

Revert "Merge remote-tracking branch 'couchbase/watson'"

This reverts commit 21cd2c5f687a378a25cfadbb0fff95036287d90e, reversing
changes made to 21991f2650ce31e787d6666268ef6a15b0bc4cfb.

Revert "Merge remote-tracking branch 'couchbase/watson'"

This reverts commit 21cd2c5f687a378a25cfadbb0fff95036287d90e, reversing
changes made to 21991f2650ce31e787d6666268ef6a15b0bc4cfb.

The merge needs to be split into two chunks - the first which fixes
the existing incorrect uses of malloc/free etc (and then other
components can also have their fixes merged), the second chunk to
actually start using cb_malloc.

Reverting back to pre-merge, so it can be split.

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

show more ...

21cd2c5f19-Sep-2016 Dave Rigby <daver@couchbase.com>

Merge remote-tracking branch 'couchbase/watson'

* couchbase/watson:
MB-20770: Switch memory tracking from {TC,je}malloc -> cbmalloc
MB-20769: Use cbmalloc for C++ new/delete allo

Merge remote-tracking branch 'couchbase/watson'

* couchbase/watson:
MB-20770: Switch memory tracking from {TC,je}malloc -> cbmalloc
MB-20769: Use cbmalloc for C++ new/delete allocations
MB-20586: [cbmalloc] With jemalloc, use system symbols for now
Revert "MB-20769: Use cbmalloc for C++ new/delete allocations"
Revert "MB-20770: Switch memory tracking from {TC,je}malloc -> cbmalloc"
MB-20770: Switch memory tracking from {TC,je}malloc -> cbmalloc
MB-20769: Use cbmalloc for C++ new/delete allocations
MB-20769: Use cbmalloc for the rest of platform
MB-20769: Use cbmalloc for cJSON
MB-20586: Add cb_malloc memory allocation API

Change-Id: Ic889b71ca7c51c283a911b82f9adfa9a3476eaa7

show more ...


08792fd518-Aug-2016 Dave Rigby <daver@couchbase.com>

MB-20769: Use cbmalloc for cJSON

cJSON performs memory allocation internally, and also allocates memory
which is returned (either directly or indirectly) to clients. For both
cases i

MB-20769: Use cbmalloc for cJSON

cJSON performs memory allocation internally, and also allocates memory
which is returned (either directly or indirectly) to clients. For both
cases it uses a set of malloc functions to perform allocation, which
default to the standard system malloc/free etc.

This patch changes the default malloc functions to be cbmalloc (the
user can still use the hook API to overide to something different). It
also fixes a couple of instances where the system functions were
incorrectly hard-coded.

Change-Id: Ibc81ee0342fdd864f653f3a816463c9c16811242
Reviewed-on: http://review.couchbase.org/67318
Well-Formed: buildbot <build@couchbase.com>
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Trond Norbye <trond.norbye@gmail.com>

show more ...

Revision tags: v4.5.0
4cee9a3919-Apr-2016 Trond Norbye <trond.norbye@gmail.com>

Refactor: Remove plugable allocation API (not used)

Use a fixed one that use malloc/calloc/strdup/free but throw
an exception if allocation fails (not all of the existing paths
check

Refactor: Remove plugable allocation API (not used)

Use a fixed one that use malloc/calloc/strdup/free but throw
an exception if allocation fails (not all of the existing paths
checked the return code which would lead to nullpointer
exceptions).

In addition to that not all methods had an API which would allow
to propagate the error back up.

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

show more ...

7067381019-Apr-2016 Trond Norbye <trond.norbye@gmail.com>

Refactor: Move cJSON to C++

This is basically moving the C code to C++ and make sure it
compiles. We'll be adding more code to make sure that
std::bad_alloc will be called etc.

Refactor: Move cJSON to C++

This is basically moving the C code to C++ and make sure it
compiles. We'll be adding more code to make sure that
std::bad_alloc will be called etc.

Change-Id: I7d197e0760659e03a9450a718fc4cf970ccfce3c
Reviewed-on: http://review.couchbase.org/63022
Tested-by: buildbot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>
Reviewed-by: Will Gardner <will.gardner@couchbase.com>

show more ...

f5ea224904-Feb-2016 Trond Norbye <trond.norbye@gmail.com>

Add to_string to convert json to string

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

Add to_string to convert json to string

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

show more ...

8238612a08-Dec-2015 Trond Norbye <trond.norbye@gmail.com>

MB-17001: refactor layout of source files

Change-Id: I7035532eaa3691286ab30e74e10e34cc0f3c2339
Reviewed-on: http://review.couchbase.org/57585
Tested-by: buildbot <build@couchbase.com

MB-17001: refactor layout of source files

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

show more ...