From fc329006a33d2661ed617bbfa6273845b79ab826 Mon Sep 17 00:00:00 2001 From: Jakub Tyszkowski Date: Wed, 26 Mar 2014 19:23:36 +0100 Subject: [PATCH] android/gatt: Remove redundant queue check It is safe to just call queue_peak_head on empty queue. Now its also safe to pass NULL to send_client_char_notify as status will be set inside it. --- android/gatt.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/android/gatt.c b/android/gatt.c index 185642790..81bfdc05d 100644 --- a/android/gatt.c +++ b/android/gatt.c @@ -1021,11 +1021,8 @@ static void discover_char_cb(uint8_t status, GSList *characteristics, cache_all_srvc_chars(characteristics, data->service->chars); - if (!queue_isempty(data->service->chars)) - send_client_char_notify(queue_peek_head(data->service->chars), + send_client_char_notify(queue_peek_head(data->service->chars), data->conn_id, data->service); - else - send_client_char_notify(NULL, data->conn_id, data->service); free(data); } -- 2.47.3