styx
/
enlightenment
forked from enlightenment/enlightenment
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

- get correct path uris 

- fix browse item action missed one case
- fix segv in selector_item_update when state has no plugin


SVN revision: 48838
This commit is contained in:
Hannes Janetzek 2010-05-14 03:11:03 +00:00
parent 57a9bdc2ed
commit 111b5b95c1
2 changed files with 70 additions and 40 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/modules/everything/evry.c
View File

@ -1035,7 +1035,8 @@ _evry_selector_item_update(Evry_Selector *sel)
} }
} }
if (!(sel->o_icon) && (EVRY_ITEM(s->plugin)->icon)) if (!(sel->o_icon) && s->plugin &&
(EVRY_ITEM(s->plugin)->icon))
{ {
o = evry_icon_theme_get(EVRY_ITEM(s->plugin)->icon, win->popup->evas); o = evry_icon_theme_get(EVRY_ITEM(s->plugin)->icon, win->popup->evas);
if (o) if (o)
@ -1304,7 +1305,10 @@ evry_browse_item(Evry_Selector *sel)
continue; continue;
if ((pp = p->browse(p, it))) if ((pp = p->browse(p, it)))
plugins = eina_list_append(plugins, pp); {
plugins = eina_list_append(plugins, pp);
}
} }
/* FIXME this is a special case for actions */ /* FIXME this is a special case for actions */
@ -1347,6 +1351,11 @@ evry_browse_item(Evry_Selector *sel)
_evry_matches_update(sel, 1); _evry_matches_update(sel, 1);
s = new_state; s = new_state;
} }
else
{
_evry_matches_update(sel, 1);
s = new_state;
}
_evry_selector_update(sel); _evry_selector_update(sel);

59
src/modules/everything/evry_util.c
View File

@ -717,24 +717,45 @@ evry_util_url_unescape(const char *string, int length)
static Eina_Bool static Eina_Bool
_isalnum(unsigned char in) _isalnum(unsigned char in)
{ {
switch (in) { switch (in)
case '0': case '1': case '2': case '3': case '4': {
case '5': case '6': case '7': case '8': case '9': case '0': case '1': case '2': case '3': case '4':
case 'a': case 'b': case 'c': case 'd': case 'e': case '5': case '6': case '7': case '8': case '9':
case 'f': case 'g': case 'h': case 'i': case 'j': case 'a': case 'b': case 'c': case 'd': case 'e':
case 'k': case 'l': case 'm': case 'n': case 'o': case 'f': case 'g': case 'h': case 'i': case 'j':
case 'p': case 'q': case 'r': case 's': case 't': case 'k': case 'l': case 'm': case 'n': case 'o':
case 'u': case 'v': case 'w': case 'x': case 'y': case 'z': case 'p': case 'q': case 'r': case 's': case 't':
case 'A': case 'B': case 'C': case 'D': case 'E': case 'u': case 'v': case 'w': case 'x': case 'y': case 'z':
case 'F': case 'G': case 'H': case 'I': case 'J': case 'A': case 'B': case 'C': case 'D': case 'E':
case 'K': case 'L': case 'M': case 'N': case 'O': case 'F': case 'G': case 'H': case 'I': case 'J':
case 'P': case 'Q': case 'R': case 'S': case 'T': case 'K': case 'L': case 'M': case 'N': case 'O':
case 'U': case 'V': case 'W': case 'X': case 'Y': case 'Z': case 'P': case 'Q': case 'R': case 'S': case 'T':
return EINA_TRUE; case 'U': case 'V': case 'W': case 'X': case 'Y': case 'Z':
default: return EINA_TRUE;
break; default:
} break;
return EINA_FALSE; }
return EINA_FALSE;
}
/* FIXME there seem to be too many ways of not following a standard.
find out which is the most popular */
static Eina_Bool
_isuric(unsigned char in)
{
switch (in)
{
case '/': case '.': case '(': case ')': case '-':
case '~': case '\'': case '_': case '@': case '+':
/* case ';': case ':':
* case '&': case '=': case '$': case ',':
* case '.': case '!':
* case '\'': */
return EINA_TRUE;
default:
break;
}
return EINA_FALSE;
} }
char * char *
@ -758,7 +779,7 @@ _evry_util_url_escape(const char *string, int inlength, int path)
in = *string; in = *string;
if (_isalnum(in) || if (_isalnum(in) ||
(path && ispunct(in))) (path && _isuric(in)))
{ {
/* just copy this */ /* just copy this */
ns[strindex++]=in; ns[strindex++]=in;