Re: MPS: dangling markers

[Gerd Möllmann]

Ihor Radchenko <yantar92@posteo.net> writes:

> Gerd Möllmann <gerd.moellmann@gmail.com> writes:
>
>> Thanks, Ihor, that looks promising. I've now also pushed a quick and
>> dirty add_marker in O(1), that's why I reverted. to get that on top.
>> Works for me...
>>
>> A comparison of how it feels compared to master would also be interesting.
>
> igc.c: In function ‘fix_marker_vector’:
> igc.c:1701:34: error: passing argument 1 of ‘NILP’ makes pointer from integer 
> without a cast [-Wint-conversion]
>  1701 |         if (NILP (v->contents[i] && !NILP (old)))
>       |                   ~~~~~~~~~~~~~~~^~~~~~~~~~~~~~
>       |                                  |
>       |                                  int
> In file included from igc.c:31:
> lisp.h:1499:21: note: expected ‘Lisp_Object’ {aka ‘struct Lisp_X *’} but 
> argument is of type ‘int’
>  1499 | (NILP) (Lisp_Object x)
>       |         ~~~~~~~~~~~~^

Soory, either I've fat-fingered something, or clang 15 is a bit too
permissive :-(

Psuhed a fix.