[Pixman] [BUG REPORT] pixman_glyph_cache_create leaks memory that it allocates

[Yuri]

On 09/18/17 03:07, Emil Velikov wrote:
>> Failure to call a destructor in C code could be interpreted as a used error,
>> or as a library error. I prefer to call it a library error, because it's
>> easier to just destroy it in the library, and not depend on users.
> One can call it spaceship if they want to. That doesn't mean it's right;-)

I see, you hold a monopoly on rightness.


> Yes there's libraries that do all sorts of things to help lazy
> users... I think pixman is not one of them.

pixman allocates and maintains the object, so it should ensure that it 
is also properly released. It doesn't appear reasonable to expect that 
many users will be always releasing it. It's easier to do this in one place.


Yuri

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/pixman/attachments/20170918/4eeb0a77/attachment.html>