2
4%
master: 4%

Ran 07 Oct 2019 12:08AM UTC

Files 648

Run time 42s

Badge

Embed ▾

Committed 06 Oct 2019 10:03PM UTC coverage: 76.976% (+0.001%) from 76.975%

Job # TEST_SUITES="docomp testtravis" ABI=32 CONFIGFLAGS=""

Build Type

push

travis-ci

Committed by

fingolfin

Commit Message

Fix BlistList for two ranges

Several results were wrong, and had been wrong since at least GAP 4.4.
E.g. before this commit:

    gap> BlistList([0..3], [-3..-1]);
    [ true, true, true, false ]

Now we get the correct result

    gap> BlistList([0,1,2,3], [-3..-1]);
    [ false, false, false, false ]

One major issue was the use of unsigned variables to store signed data. Two
variables were only of type `long` instead of `Int`, but on 64bit, this could
lead to further issues.

The logic handling the intersection was broken and overly complicated. The
rewritten logic should be simpler.

Finally, there is a tiny optimization enabled by switching from 1-based
indexing to 0-based.

Run Details

242376 of 314871 relevant lines covered (76.98%)

4834369.03 hits per line

gap-system / gap / 10736 / 2
4%
master: 4%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 10736.2 (TEST_SUITES="docomp testtravis" ABI=32 CONFIGFLAGS="")

gap-system / gap / 10736 / 2 4% master: 4%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 10736.2 (TEST_SUITES="docomp testtravis" ABI=32 CONFIGFLAGS="")

gap-system / gap / 10736 / 2
4%
master: 4%

README BADGES
x