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

efreet - work around conconforming apps putting startupwmclass in quotes 

riot does

StartupWMClass="Riot"

https://specifications.freedesktop.org/desktop-entry-spec/desktop-entry-spec-latest.html

does not allow for quoting here. you can quote fields in an exec line
- for shell and replacement purposes, but not other fields like this
above. this is a workaround a broken app so matching window to desktop
file works.

@fix
This commit is contained in:
Carsten Haitzler 2020-04-25 14:44:23 +01:00
parent 7f984ce4a5
commit 680dee1daf
1 changed files with 16 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/lib/efreet/efreet_desktop.c
View File

@ -777,7 +777,22 @@ efreet_desktop_application_fields_parse(Efreet_Desktop *desktop, Efreet_Ini *ini
if (val) desktop->path = strdup(val);
val = efreet_ini_string_get(ini, "StartupWMClass");
if (val) desktop->startup_wm_class = strdup(val);
if ((val) && (val[0]) && (val[1]))
{
size_t len = strlen(val);
if (((val[0] == '"') && (val[len - 1] == '"') ) ||
((val[0] == '\'') && (val[len - 1] == '\'')))
{
// fixup for some spec-violating apps that put startupwmclass
// in quotes... spec doesnt allow for this. just escapes.
char *tmpval = alloca(len - 1);
strncpy(tmpval, val + 1, len - 2);
tmpval[len - 2] = '\0';
desktop->startup_wm_class = strdup(tmpval);
}
else
desktop->startup_wm_class = strdup(val);
}
val = efreet_ini_string_get(ini, "Categories");
if (val)