gryf/wmaker
1
0
Fork 0
mirror of https://github.com/gryf/wmaker.git synced 2025-12-20 04:48:06 +01:00
Code Issues Projects Releases Wiki Activity

wmaker: removed unnecessary assignation (Coverity #50257)

Browse Source 
As pointed by Coverity, the return value is saved into a variable, but this
value is never used. As the variable is re-used afterwards, this
assignation could mislead on what is done, so for code maintainability the
value is just ignored.

Signed-off-by: Christophe CURIS <christophe.curis@free.fr>
Signed-off-by: Carlos R. Mafra <crmafra@gmail.com>
This commit is contained in:
Christophe CURIS
2014-11-15 19:40:39 +01:00
committed by Carlos R. Mafra
parent 3d9c059591
commit 572af55c9f
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/client.c
View File

@@ -574,6 +574,7 @@ void wClientCheckProperty(WWindow * wwin, XPropertyEvent * event)
if (foo->fake_group && foo->fake_group == fPtr) { if (foo->fake_group && foo->fake_group == fPtr) {
WSETUFLAG(foo, shared_appicon, 0); WSETUFLAG(foo, shared_appicon, 0);
foo->fake_group = NULL; foo->fake_group = NULL;
if (foo->group_id != None) if (foo->group_id != None)
foo->main_window = foo->group_id; foo->main_window = foo->group_id;
else if (foo->client_leader != None) else if (foo->client_leader != None)
@@ -582,9 +583,9 @@ void wClientCheckProperty(WWindow * wwin, XPropertyEvent * event)
foo->main_window = foo->client_win; foo->main_window = foo->client_win;
else else
foo->main_window = None; foo->main_window = None;
if (foo->main_window) {
wapp = wApplicationCreate(foo); if (foo->main_window)
} wApplicationCreate(foo);
} }
foo = foo->prev; foo = foo->prev;
} }