Hello!

Seems there is a problem in

ef_icon_theme.c:590

        if (!strcmp(p, ".png") || !strcmp(p, ".xpm"))
        {
            *p = '\0';

            p = strrchr(file, '/');
            if (p) p++;
            if (p) ecore_hash_set(icons, strdup(p), strdup(p));
        }

I suppose there is a bug here, and the code should be like follows:

        if (!strcmp(p, ".png") || !strcmp(p, ".xpm"))
        {
            *p = '\0';

            p = strrchr(file, '/');
            if (p) p++;
            else p = file;
            if (p) ecore_hash_set(icons, strdup(p), strdup(p));
        }

It is a small notice however and we (me and Vincent Torri) cannot decide 
whether this is a feature or bug. So just a warning for you:). Tests pass good 
and without it actually, but "icons" list stays empty while it should be 
definitely filled.

Regards,
Dzmitry Mazouka


-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to