Re: Merging bignum to master

[Noam Postavsky <npostavs@gmail.com>]

On 13 August 2018 at 18:58, Paul Eggert <eggert@cs.ucla.edu> 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);
          ^~~~~~~