[Pixman] [PATCH 0/8] Add glyph API

[Søren Sandmann]

Behdad Esfahbod <behdad at behdad.org> writes:

> On 05/30/2012 07:41 PM, Søren Sandmann wrote:
>> While this is going on, the cache must be "frozen", which basically
>> means that glyphs will not be evicted from it. That way, the
>> application can be sure that all the glyphs are available at
>> compositing time. Afterwards, when the cache is "thawed", it may
>> decide to evict some glyphs if the cache is too full.
>
> This, with the current code, can cause in infinite loops if an app freezes
> then adds too many glyphs (32k?).  Right?

No, there is a check in pixman_glyph_cache_insert() that makes it return
NULL in that case. Ie., it will look to the application as an
out-of-memory condition, but there won't be any infinite loops.


Soren