Re: Some vars now limited to fixnum size. (Was: Merging bignum to master)

[Pip Cet]

On Mon, Aug 20, 2018 at 6:01 PM Eli Zaretskii <address@hidden> wrote:
> > From: Paul Eggert <address@hidden>
> > Date: Mon, 20 Aug 2018 10:27:34 -0700
> > Cc: address@hidden, Pip Cet <address@hidden>, address@hidden
> >
> > +  if (FIXNUMP (n))
> > +    num = XFIXNUM (n);
> > +  else
> > +    {
> > +      num = mpz_sgn (XBIGNUM (n)->value);
> > +      if (0 < num)
> > +     num = EMACS_INT_MAX;  /* LIST cannot possibly be this long.  */
>
> So we now silently replace the argument with another, smaller value,
> and then go ahead as if business as usual?  Is that a good,
> user-friendly behavior?

It's certainly incorrect for circular lists.