ICRAR / ijson / 231

100%

master: 100%

push

travis-ci-com

Create builder_t objects on the stack, not heap

We previously created the builder separately on the heap. This process
involved memory allocations, which have a chance to fail, although in
our case it is very minimal. This small possibility however accounted
for some code that was missing coverage (because the error never
occurred).

This code changes how builder_t objects are managed, allocating its
memory together with the object that contains it (ItemsBasecoro and
KVItemsBasecoro). This in turn means we don't need to perform an extra
memory allocation, and therefore removing this uncovered code.

This change should also have some further improvements, as the builder's
memory resides closer potentially to the rest of its container object.

Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>

21 of 21 new or added lines in 3 files covered. (100.0%)

1514 of 1517 relevant lines covered (99.8%)

369.2 hits per line