Skip to content

Commit

Permalink
Changed InfoWarning -> InfoLattice in one Info message
Browse files Browse the repository at this point in the history 
'Alternating recognition needed' was in InfoWarning while the rest
of the messages in this file - in InfoLattice. This seems to be
done accidentally. The InfoWarning message was displayed when GAP
was started with -A option and this broke regression tests.
  • Loading branch information
Alexander Konovalov committed Nov 28, 2016
1 parent 5ca402f commit 488af13
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/maxsub.gi
View file
Original file line number Diff line number Diff line change
Expand Up @@ -542,7 +542,7 @@ local id,m,epi;
# following is stopgap for L
id:=DataAboutSimpleGroup(G);
if id.idSimple.series="A" then
Info(InfoWarning,1,"Alternating recognition needed!");
Info(InfoLattice,1,"Alternating recognition needed!");
elif id.idSimple.series="L" then
m:=ClassicalMaximals("L",id.idSimple.parameter[1],id.idSimple.parameter[2]);
if m<>fail then
Expand Down

2 comments on commit 488af13

@hulpke
Copy link
Contributor

@hulpke hulpke commented on 488af13 Nov 28, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The different info class was deliberate -- without alternating group recognition available, nor predefined date from the TOM available, this will a significant performance hit (of taking hours or longer instead of seconds). I will change this to InfoPerformance in my next PR.

@olexandr-konovalov
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hulpke thanks for clarification - please do change it then!

Please sign in to comment.