[E-devel] Efreet bin ef_icon theme bug

Top Page
Attachments:
Message as email
+ (text/plain)
Delete this message
Author: Dmitriy Mazovka
Date:  
To: enlightenment-devel
Subject: [E-devel] Efreet bin ef_icon theme bug
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
enlightenment-devel@???
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel