[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Some vars now limited to fixnum size. (Was: Merging bignum to master
From: |
Pip Cet |
Subject: |
Re: Some vars now limited to fixnum size. (Was: Merging bignum to master) |
Date: |
Mon, 20 Aug 2018 19:55:37 +0000 |
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.
- Re: Merging bignum to master, (continued)
- Re: Merging bignum to master, Paul Eggert, 2018/08/14
- Re: Merging bignum to master, Pip Cet, 2018/08/15
- Re: Merging bignum to master, Paul Eggert, 2018/08/15
- Re: Merging bignum to master, Andy Moreton, 2018/08/15
- Re: Merging bignum to master, Stefan Monnier, 2018/08/16
- Some vars now limited to fixnum size. (Was: Merging bignum to master), Karl Fogel, 2018/08/20
- Re: Some vars now limited to fixnum size. (Was: Merging bignum to master), Paul Eggert, 2018/08/20
- Re: Some vars now limited to fixnum size. (Was: Merging bignum to master), Eli Zaretskii, 2018/08/20
- Re: Some vars now limited to fixnum size. (Was: Merging bignum to master), Paul Eggert, 2018/08/20
- Re: Some vars now limited to fixnum size. (Was: Merging bignum to master), Eli Zaretskii, 2018/08/20
- Re: Some vars now limited to fixnum size. (Was: Merging bignum to master),
Pip Cet <=
- Re: Some vars now limited to fixnum size. (Was: Merging bignum to master), Paul Eggert, 2018/08/20
- Re: Some vars now limited to fixnum size., Tom Tromey, 2018/08/21
- Re: Some vars now limited to fixnum size., Andy Moreton, 2018/08/21
- Re: Some vars now limited to fixnum size., Paul Eggert, 2018/08/21
- Re: Some vars now limited to fixnum size., Tom Tromey, 2018/08/22
- Re: Some vars now limited to fixnum size. (Was: Merging bignum to master), Richard Stallman, 2018/08/20
- Re: Some vars now limited to fixnum size. (Was: Merging bignum to master), Paul Eggert, 2018/08/21
- Re: Some vars now limited to fixnum size. (Was: Merging bignum to master), Richard Stallman, 2018/08/22
- Re: Some vars now limited to fixnum size. (Was: Merging bignum to master), Paul Eggert, 2018/08/22
- Re: Some vars now limited to fixnum size. (Was: Merging bignum to master), Eli Zaretskii, 2018/08/20