ICRAR / ijson / 158 / 5
99%
master: 100%

LAST BUILD BRANCH: release-3.2.0
DEFAULT BRANCH: master
Ran
Files 33
Run time 2s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 99.333%. First build
UPLOAD_C_COVERAGE=yes

push

travis-ci-com

rtobar 
Reintroduce common.* functions

This commit re-introduces the common.* functions that were removed
during the redesign of ijson 3. In ijson 2.X these functions were the
building blocks over which the backend-specific functions were built
upon. However, with the redesign of the library in version 3 they were
not needed anymore and they were removed.

It turns out users around the globe used these methods, as they iterated
over events in ways we didn't anticipate. Thus, there is value in adding
these functions back, even if it's not in a fully-performant way. With
time we can think of moving these functions into each backend so they
use the backend's underlying function, which is important for
performance.

Tests have been added to double-check that the behavior is the expected
one. In the case of items and kvitems we event skip through a few events
before reading the rest, which cannot be safely done with parse, as its
state machine depends on reading all events in order.

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

# Please enter the commit message for your changes. Lines starting
# with '#' will be ignored, and an empty message aborts the commit.
#
# On branch master
# Your branch is up to date with 'origin/master'.
#
# Changes to be committed:
#	modified:   ijson/common.py
#	modified:   tests.py
#
# Untracked files:
#	dump.py
#	lala.py
#	notes/compare results.ipynb
#

596 of 600 relevant lines covered (99.33%)

529.55 hits per line

Source Files on job 158.5 (UPLOAD_C_COVERAGE=yes)