[PUSHED v2] fdo#42779: Implement icon theme lookaside directory

Gábor Stefanik netrolller.3d at gmail.com
Mon Apr 30 15:34:01 PDT 2012

On Mon, Apr 30, 2012 at 1:24 PM, Michael Meeks <michael.meeks at suse.com> wrote:
> Hi Gabor,
> On Sun, 2012-04-29 at 04:56 +0200, Gábor Stefanik wrote:
>> This enables overriding icons found in an images.zip file with ones in
>> a directory named "images" next to the zip file.
>        Looks lovely, as/when there are dev-builds we should poke the artists
> about it so they know it's there & get them excited about it :-)
>> Icon caching is also disabled when a lookaside directory is detected,
>> so changes to the icons in the directory are applied immediately.
>        So - one minor problem here is that, we do this system-call thrash for
> each icon for a file that is not there :-) I'd rather prefer that we
> require the images/ directory to exist at startup - if it is there, we
> always do the lookups first in there and turn the cache off - otherwise,
> we never look in there.
>        Any chance of a minor follow-on tweak to do that ? the strace of a
> normal startup without that directory should not have scads of opens
> of .pngs.

Will post a patch for this, thanks for the heads up.

>> Icons in a lookaside directory currently always take precedence over
>> ones in zip files.
>        Thanks for that, can you close the easy-hack bug too ? :-)

Will do.

>        Nice work,
>                Michael.
> --
> michael.meeks at suse.com  <><, Pseudo Engineer, itinerant idiot


More information about the LibreOffice mailing list