Ilya Zakharevich on Thu, 25 Oct 2001 13:17:43 -0400

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

Re: math::pari + pari-2.1.2

On Thu, Oct 25, 2001 at 06:57:38PM +0200, Karim BELABAS wrote:
> > Junk in, junk out.  This is a Perl module; if the Perl configuration
> > is busted bad enough, one is not able to build a Perl module, period.
> > Nothing specific to Math::Pari.  I do not think I *want* to treat this
> > case specially.
> Sure. A quick mention of the symptoms in the README would be user-friendly.

Unfortunately, the symptoms may be arbitrary...  I already added `are
you crazy?' section in INSTALL, though. ;-)

> > BTW, any reason why ulong is *still* used in the sources?  I sent a
> > recipe how to fix it around 5 years ago.  IIRC, it was something as
> > simple as a global edit as in
> >
> >   pfind src '=~ s/\bulong\b/PARI_ulong/'
> Well, it's a (nearly) standard type name.

Exactly.  One should avoid using "(nearly) standard" stuff as a plague.

> Also it's not that simple, it would make a lot of sense to have avma and
> friends be of type e.g PARI_stack_ptr, whereas the arithmetic word operations
> should operate on e.g PARI_word [these two changes would remove zillions of
> Warnings from picky compilers, btw] etc... It doesn't make much sense to make
> a global edit which would not clarify things, falling back to (unsigned long)
> would do just as well.

Nope.  To build a 64-bit PARI on a 32-bit-ILP compiler you need to be
able to specify your own types to the components of PARI objects.
What I said is just one step in this direction.

> > > Pari.xs:224: warning: static declaration for `changevalue' follows non-static
> >
> > Thanks.  Cannot find it in CHANGES, when was it made global?  It is
> > not in 2.0.15 yet...
> anal.c revision 1.55 (2001/09/18 11:04:50) that means just after
> release-2.2.0.  It was a _huge_ patch [ introduced writebin() ], I forgot to
> mention the globalization in CHANGES (also in 'cvs log'...).

Thanks, I will insert a guard for PARI_VERSION (sp?) <= 2002000.