gap-system / gap / 11011

4%

push

travis-ci

Fix bug in IrrConlon leading to unexpected errors

SubgroupNC expects a parent plus a list of generators of the subgroup.
However, the old code actually passed a subgroup as "list of generators".
Which is bad by itself, but it also means that later `Length( gens )` failed.
This is resolved by the new code, which also ensures that known attributes of
the subgroups, such as its size, are retained.

This used to work in GAP 4.9, but in GAP 4.10 we changed the code which
creates subgroups, which introduced this regression in behavior.

Reported by Benjamin Sambale.

2 of 2 new or added lines in 1 file covered. (100.0%)

285779 of 337459 relevant lines covered (84.69%)

29998060.68 hits per line