[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Merging bignum to master
From: |
Noam Postavsky |
Subject: |
Re: Merging bignum to master |
Date: |
Mon, 13 Aug 2018 21:12:01 -0400 |
On 13 August 2018 at 18:58, Paul Eggert <address@hidden> wrote:
> meantime I installed the attached first patch to fix a glitch I saw in this
> area while looking at the current XFIXNUMPTR uses.
> --- a/src/lisp.h
> +++ b/src/lisp.h
> @@ -1189,9 +1189,15 @@ XFIXNUMPTR (Lisp_Object a)
> }
>
> INLINE Lisp_Object
> +make_pointer_integer_unsafe (void *p)
> +{
> + return TAG_PTR (Lisp_Int0, p);
> +}
> +
> +INLINE Lisp_Object
> make_pointer_integer (void *p)
> {
> - Lisp_Object a = TAG_PTR (Lisp_Int0, p);
> + Lisp_Object a = make_pointer_integer_unsafe (p);
I use ./configure --enable-check-lisp-object-type, and this gives me
CC dispnew.o
In file included from dispnew.c:27:0:
lisp.h: In function ‘make_pointer_integer_unsafe’:
lisp.h:568:28: error: expected expression before ‘{’ token
# define LISP_INITIALLY(w) {w}
^
lisp.h:790:3: note: in expansion of macro ‘LISP_INITIALLY’
LISP_INITIALLY ((Lisp_Word) ((untagged_ptr) (ptr) + LISP_WORD_TAG (tag)))
^~~~~~~~~~~~~~
lisp.h:1194:10: note: in expansion of macro ‘TAG_PTR’
return TAG_PTR (Lisp_Int0, p);
^~~~~~~
- Re: Merging bignum to master, (continued)
- Re: Merging bignum to master, Stefan Monnier, 2018/08/16
- Re: Merging bignum to master, Ulrich Mueller, 2018/08/12
- Re: Merging bignum to master, Eli Zaretskii, 2018/08/12
- Re: Merging bignum to master, Ulrich Mueller, 2018/08/12
- Re: Merging bignum to master, Paul Eggert, 2018/08/12
Re: Merging bignum to master, John Wiegley, 2018/08/12
Re: Merging bignum to master, Pip Cet, 2018/08/12
- Re: Merging bignum to master, Tom Tromey, 2018/08/12
- Re: Merging bignum to master, Paul Eggert, 2018/08/13
- Re: Merging bignum to master,
Noam Postavsky <=
- Re: Merging bignum to master, Pip Cet, 2018/08/14
- 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