homalg-project / CddInterface

Gap Interface to Cdd
https://homalg-project.github.io/CddInterface
Other
1 stars 5 forks source link

Don't call `Print` in the toplevel of .gi / .gd files, modify package banner instead. #13

Closed fingolfin closed 5 years ago

fingolfin commented 5 years ago

I just noticed in this polyhedra.gi:

Print( "Interface to cddlib: Implementation of the double description method of Motzkin et al.\n");
Print( "   Copyright (C) 1996, Komei Fukuda, fukuda@ifor.math.ethz.ch\n" );
Print( "   http://www.ifor.math.ethz.ch/~fukuda/cdd_home/cdd.html\n" );

Please don't do that! Such messages should be printed as part of the package banner, so that e.g. tests can disable them.