raster pushed a commit to branch master. http://git.enlightenment.org/core/efl.git/commit/?id=2f4fb3edac63413ed292ebca43bda79bfea378cb
commit 2f4fb3edac63413ed292ebca43bda79bfea378cb Author: Carsten Haitzler (Rasterman) <ras...@rasterman.com> Date: Sun Jul 5 12:53:14 2015 +0900 efreet - improve edstop exec find to use the command with less args efreet will just match any desktop file (at random basically - whatever is first in the list), thjat has the base command looked at. if you have various desktip files with the same command, like: mycommand mycommand %U mycommand -a -b -c /path/to/file /usr/bin/mycommand etc. - which one does it match? it'll pick the first and the list ordering is arbitrary, so this is pretty bad. this strips off the base command at the start (full path or whatever) and then uses the command with the shortest argument length. this means the generic command is found first if we look for "mycommand" in the above list, whihc ends up the best... ESPECIALLY for steam which does just this. --- src/lib/efreet/efreet_utils.c | 111 ++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 107 insertions(+), 4 deletions(-) diff --git a/src/lib/efreet/efreet_utils.c b/src/lib/efreet/efreet_utils.c index 809d738..ae3ab17 100644 --- a/src/lib/efreet/efreet_utils.c +++ b/src/lib/efreet/efreet_utils.c @@ -5,6 +5,7 @@ /* TODO: add no_display check, as we might want only displayable items */ #include <fnmatch.h> +#include <ctype.h> #include <Ecore_File.h> @@ -170,11 +171,80 @@ efreet_util_desktop_file_id_find(const char *file_id) return ret; } +static char * +efreet_util_cmd_args_get(const char *cmd) +{ + Eina_Strbuf *buf; + char *args; + const char *p; + Eina_Bool in_qout_double = EINA_FALSE; + Eina_Bool in_qout_single = EINA_FALSE; + Eina_Bool atargs = EINA_FALSE; + Eina_Bool ingap = EINA_FALSE; + + buf = eina_strbuf_new(); + if (!buf) return NULL; + + // get the arguments to the command as a string on its own + for (p = cmd; *p; p++) + { + if (!atargs) + { + if (in_qout_double) + { + if (*p == '\\') + { + if (p[1]) p++; + } + else if (*p == '"') in_qout_double = EINA_FALSE; + } + else if (in_qout_single) + { + if (*p == '\\') + { + if (p[1]) p++; + } + else if (*p == '\'') in_qout_single = EINA_FALSE; + } + else + { + if (*p == '\\') + { + if (p[1]) p++; + } + else if (*p == '"') in_qout_double = EINA_TRUE; + else if (*p == '\'') in_qout_single = EINA_TRUE; + else + { + if (isspace((unsigned char)(*p))) + { + atargs = EINA_TRUE; + ingap = EINA_TRUE; + } + } + } + } + else + { + if (ingap) + { + if (!isspace((unsigned char)(*p))) ingap = EINA_FALSE; + } + if (!ingap) eina_strbuf_append_char(buf, *p); + } + } + + args = eina_strbuf_string_steal(buf); + eina_strbuf_free(buf); + if (!args) return strdup(""); + return args; +} + EAPI Efreet_Desktop * efreet_util_desktop_exec_find(const char *exec) { Efreet_Cache_Hash *hash = NULL; - Efreet_Desktop *ret = NULL; + Efreet_Desktop *ret = NULL, *bestret = NULL; Efreet_Cache_Array_String *names = NULL; unsigned int i; @@ -207,11 +277,44 @@ efreet_util_desktop_exec_find(const char *exec) for (j = 0; j < array->array_count; j++) { ret = efreet_desktop_get(array->array[j]); - if (ret) break; + if (ret) + { + if (!bestret) bestret = ret; + else + { + if (ret->exec) + { + // perfect match - best + if (!strcmp(ret->exec, exec)) + { + bestret = ret; + goto done; + } + else if (bestret->exec) + { + char *f1, *f2; + + f1 = efreet_util_cmd_args_get(ret->exec); + f2 = efreet_util_cmd_args_get(bestret->exec); + if ((f1) && (f2)) + { + // if this is shorter (less arguments) than best + // match so far, thewn this is the best match + if (strlen(f1) < strlen(f2)) + { + bestret = ret; + } + } + free(f1); + free(f2); + } + } + } + } } - if (ret) break; } - return ret; +done: + return bestret; } EAPI Efreet_Desktop * --