Re: [Emacs-diffs] trunk r117846: Add macros to allocate temporary Lisp objects with alloca.

[Dmitry Antipov]

On 09/10/2014 10:43 AM, Paul Eggert wrote:

> I installed some changes that I hope improve things.

| (pointer_valid_for_lisp_object): Remove.  This check is not
| necessary, since make_lisp_ptr is already doing it.  All uses removed.

This is still in use in verify_alloca:

../../trunk/src/alloc.c: In function ‘verify_alloca’:
../../trunk/src/alloc.c:7179:7: error: implicit declaration of function 
‘pointer_valid_for_lisp_object’ [-Werror=implicit-function-declaration]
       eassert (pointer_valid_for_lisp_object (ptr));
       ^
../../trunk/src/alloc.c:7179:7: error: nested extern declaration of 
‘pointer_valid_for_lisp_object’ [-Werror=nested-externs]
cc1: all warnings being treated as errors

If you think that this function is redundant, feel free to remove it; otherwise 
let's use make_lisp_ptr.

Dmitry