Don't need the return variable here.

Don't free the title.



SVN revision: 46146
This commit is contained in:
Christopher Michael 2010-02-13 16:19:41 +00:00
parent 820d76b53d
commit b1804a21eb
1 changed files with 4 additions and 14 deletions

View File

@ -36,14 +36,12 @@ e_illume_border_is_keyboard(E_Border *bd)
if (il_cfg->policy.vkbd.match.title)
{
char *title;
int ret = 0;
title = bd->client.netwm.name;
if (!title) title = bd->client.icccm.title;
if ((title) &&
(!strcmp(title, il_cfg->policy.vkbd.title)))
ret = 1;
if (ret) return ret;
return 1;
}
if (il_cfg->policy.vkbd.match.name)
{
@ -77,9 +75,7 @@ e_illume_border_is_bottom_panel(E_Border *bd)
if (!title) title = bd->client.icccm.title;
if ((title) &&
(!strcmp(title, il_cfg->policy.softkey.title)))
ret = 1;
if (title) free(title);
if (ret) return ret;
return 1;
}
if (il_cfg->policy.softkey.match.name)
{
@ -105,15 +101,12 @@ e_illume_border_is_top_shelf(E_Border *bd)
if (il_cfg->policy.indicator.match.title)
{
char *title;
int ret = 0;
title = bd->client.netwm.name;
if (!title) title = bd->client.icccm.title;
if ((title) &&
(!strcmp(title, il_cfg->policy.indicator.title)))
ret = 1;
if (title) free(title);
if (ret) return ret;
return 1;
}
if (il_cfg->policy.indicator.match.name)
{
@ -136,15 +129,12 @@ e_illume_border_is_home(E_Border *bd)
if (il_cfg->policy.home.match.title)
{
char *title;
int ret = 0;
title = bd->client.netwm.name;
if (!title) title = bd->client.icccm.title;
if ((title) &&
(!strcmp(title, il_cfg->policy.home.title)))
ret = 1;
if (title) free(title);
if (ret) return ret;
return 1;
}
if (il_cfg->policy.home.match.name)
{