Bill Allombert on Wed, 26 Oct 2016 00:24:59 +0200

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

Re: Backwards-incompatible change to idealappr()

On Tue, Oct 25, 2016 at 09:25:09AM +0200, Karim Belabas wrote:
> * Jeroen Demeyer [2016-10-24 22:06]:
> > In PARI-2.8.1 which was just released, idealappr() no longer takes a "flag"
> > argument. I thought that PARI/GP cared a lot about backwards compatibility,
> > so it is surprising to remove this flag, which was never even deprecated. So
> > it would be more logical to keep the flag, but deprecate it (like the flag
> > in matkerint).
> Indeed, thanks for catching this one: the library function idealappr0
> had been kept but I mistakenly changed C-Name from idealappr0 to
> idealappr (and the attached Prototype) in the description file. Fixed in commit

A similar issue is with nfbasis(,1) in alnuth:

The code does 
  if(type(version)!="t_POL" && lex(version(),[2,4,3])>=0,
    fac = lift(nffactor(f, pol ))
  , if(poldegree(pol)*3<n,
      fac = lift(factornf(pol,f));
      nf = nfinit([f, nfbasis(f,1)]);
      fac = lift(nffactor(nf, pol ))

so actually nfbasis(f,1) is never executed, unfortunately this leads to a
compilation error.

  ***   too many arguments: nfbasis(f,1)