From f59bdb51b16d2a3f8dbff7705c8534d94818ebdc Mon Sep 17 00:00:00 2001 From: Jaganath Kanakkassery Date: Mon, 1 Apr 2013 15:08:00 +0530 Subject: [PATCH] attrib: Remove norequests and noresponses variables Passing NULL is fine to g_queue_is_empty(), so removing these variables makes the code more readable --- attrib/gattrib.c | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/attrib/gattrib.c b/attrib/gattrib.c index 01c19f948..f95f2fba9 100644 --- a/attrib/gattrib.c +++ b/attrib/gattrib.c @@ -394,7 +394,6 @@ static gboolean received_data(GIOChannel *io, GIOCondition cond, gpointer data) uint8_t buf[512], status; gsize len; GIOStatus iostat; - gboolean norequests, noresponses; if (attrib->stale) return FALSE; @@ -447,11 +446,6 @@ static gboolean received_data(GIOChannel *io, GIOCondition cond, gpointer data) status = 0; done: - norequests = attrib->requests == NULL || - g_queue_is_empty(attrib->requests); - noresponses = attrib->responses == NULL || - g_queue_is_empty(attrib->responses); - if (cmd) { if (cmd->func) cmd->func(status, buf, len, cmd->user_data); @@ -459,7 +453,8 @@ done: command_destroy(cmd); } - if (!norequests || !noresponses) + if (!g_queue_is_empty(attrib->requests) || + !g_queue_is_empty(attrib->responses)) wake_up_sender(attrib); return TRUE; -- 2.47.3