As pointed by Coverity, the pointer in wwin->frame have already been dereferenced many times in the function, so it is useless to include a check later; removing it makes the code smaller thus easier to maintain.
Signed-off-by: Christophe CURIS <[email protected]> --- src/actions.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/actions.c b/src/actions.c index 4e87746..1d46b03 100644 --- a/src/actions.c +++ b/src/actions.c @@ -1155,7 +1155,7 @@ void wIconifyWindow(WWindow *wwin) const char *title; char title_buf[32]; - if (wwin->frame && wwin->frame->title) { + if (wwin->frame->title) { title = wwin->frame->title; } else { snprintf(title_buf, sizeof(title_buf), "(id=0x%lx)", wwin->client_win); -- 2.1.4 -- To unsubscribe, send mail to [email protected].
