From a93570e5be7c6a2493bc8555da4a31e889de9210 Mon Sep 17 00:00:00 2001 From: Christophe CURIS Date: Sat, 25 Apr 2015 12:44:26 +0200 Subject: [PATCH] wmaker: remove useless null pointer check (Coverity #109612) 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 --- src/actions.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/actions.c b/src/actions.c index 4e877465..1d46b035 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);