gap-system / gap / 9320

4%

Build Type

push

travis-ci

Committed by fingolfin

Commit Message

vec8bit: refactor, add struct FieldInfo8Bit

We sacrifice some memory efficiency in order to make the access to the
8bit fieldinfo data used by vec8bit.c more readable and debuggable. The
drawback is that our memory usage increases. In the best case, when no
compressed vectors get created at all, the memory overhead is zero The
worst case happens if compressed vectors of all field sizes <= 256 are
created by the user in one session (which should happen rarely for most
people). With it, the overall size of the FieldInfo8Bit global plist
goes from ~17 MB up to ~44 MB.

This also removes a workaround for a compiler bug in Xcode 5.0 that
could lead a crash in the compiled code. However, as the whole code was
restructured, that crash should still be effectively fixed; plus
hopefully nobody is using those ancient compilers anymore.

Run Details

79 of 99 new or added lines in 2 files covered. (79.8%)

201 existing lines in 1 file now uncovered.

289147 of 339884 relevant lines covered (85.07%)

22220019.26 hits per line