bug#41200: Displaying a tooltip with x-show-tip gets very slow as more faces are defined

[Clément Pit-Claudel]

On 15/05/2020 15.38, Eli Zaretskii wrote:
>> Cc: 41200@debbugs.gnu.org, juri@linkov.net
>> From: Clément Pit-Claudel <cpitclaudel@gmail.com>
>> Date: Fri, 15 May 2020 15:23:00 -0400
>>
>>>>> What is the default size of a hash-table?
>>>>
>>>> 65 entries, I think.
>>>
>>> The number of basic faces is just 18.  Is 65 a good initial size for
>>> that?
>>
>> I think that map stores more than just the basic faces: in fact, $ emacs -Q 
>> --batch --eval '(print (length (frame-face-alist)))' prints 100, so maybe I 
>> should even make the map larger by default?
> 
> We are miscommunicating.  I was talking about the size before loadup,
> i.e. in temacs when it starts up.

Ah, yes.  I think I mas mixing up the face_hash_table of each frame and 
Vface_new_frame_defaults.  Attached is a patch that makes 
Vface_new_frame_defaults 33-entries large.  Should I also change the default 
size for face_hash_table in struct frame?

> Later, when we load preloaded packages, the size should grow, but the
> hash-table takes care of that by itself, doesn't it?
Yes, definitely.

0001-Store-frame-faces-in-hash-tables-instead-of-alists.patch
Description: Text Data