On Sat, 15 May 2010, Enlightenment SVN wrote:

> Log:
>  add FIXME
>
> Author:       jeffdameth
> Date:         2010-05-15 09:57:28 -0700 (Sat, 15 May 2010)
> New Revision: 48893
>
> Modified:
>  trunk/efreet/src/lib/efreet_icon.c
>
> Modified: trunk/efreet/src/lib/efreet_icon.c
> ===================================================================
> --- trunk/efreet/src/lib/efreet_icon.c        2010-05-15 16:29:07 UTC (rev 
> 48892)
> +++ trunk/efreet/src/lib/efreet_icon.c        2010-05-15 16:57:28 UTC (rev 
> 48893)
> @@ -115,6 +115,9 @@
> {
>     if (!efreet_icon_themes)
>     {
> +        /* FIXME when svg and xpm with same name exist but svg
> +        loader was not built xpm icon will not be found.
> +        use sth like #ifdef EVAS_IMAGE_LOADER_SVG when possible. */

I don't think that using #ifdef is a good solution as the svg loader can 
be installed (by a package installer) after efreet has been compiled. This 
should be done at runtime and not at build time

Vincent

>         const char *default_exts[] = {".png", ".svg", ".xpm", NULL};
>         int i;
>
>
>
> ------------------------------------------------------------------------------
>
> _______________________________________________
> enlightenment-svn mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/enlightenment-svn
>
>

------------------------------------------------------------------------------

_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to