emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: scratch/igc abefefd4f88: Add a staticpro in search.c (bug#75459)


From: Stefan Kangas
Subject: Re: scratch/igc abefefd4f88: Add a staticpro in search.c (bug#75459)
Date: Fri, 10 Jan 2025 19:55:00 +0000

Gerd Möllmann <gerd.moellmann@gmail.com> writes:

> Stefan Kangas <stefankangas@gmail.com> writes:
>
>>> diff --git a/src/search.c b/src/search.c
>>> index 36a932b015b..98d11131e31 100644
>>> --- a/src/search.c
>>> +++ b/src/search.c
>>> @@ -3446,6 +3446,11 @@ syms_of_search (void)
>>>        staticpro (&searchbufs[i].regexp);
>>>        staticpro (&searchbufs[i].f_whitespace_regexp);
>>>        staticpro (&searchbufs[i].syntax_table);
>>> +#ifdef HAVE_MPS
>>> +      /* Strictly speaking, this is also necessary for the old GC,
>>> +    unless one would prove it isn't.  Don't rock the boat.  */
>>> +      staticpro (&searchbufs[i].buf.translate);
>>> +#endif
>>
>> Is there any harm from doing this for the old GC also?
>
> No :-)

Thanks, that is my understanding also, so I made this unconditional.
It seems both simpler and more correct.

Maybe we should cherry-pick it to master even, but I'll leave
that to someone else.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]