Don't use GAPInfo.PackagesLoaded

Issue #18 resolved
Max Horn created an issue

Right now, there is code like this in fining:

    if not "grape" in RecNames(GAPInfo.PackagesLoaded) then
       Error("You must load the Grape package in order to use IncidenceGraph.\n");
    fi;

But note that GAPInfo.PackagesLoaded is not documented. The equivalent way using documented APIs is this:

    if not IsPackageMarkedForLoading("grape", "") then
       Error("You must load the Grape package in order to use IncidenceGraph.\n");
    fi;

In GAP 4.10, there will also be a function IsPackageLoaded, which performs an additional check and for which the second (version) argument is optional, so that then you could also write:

    if not IsPackageLoaded("grape") then
       Error("You must load the Grape package in order to use IncidenceGraph.\n");
    fi;

Comments (1)

  1. Log in to comment