From 07b33109c0a39ff11c1facd030d43c6686fd7dc2 Mon Sep 17 00:00:00 2001 From: Szymon Janc Date: Fri, 23 Jan 2015 18:25:27 +0100 Subject: [PATCH] android/gatt: Use switch instead of if-else for enum It is more commomn to use switch instead of if-else when handling enums. --- android/gatt.c | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/android/gatt.c b/android/gatt.c index 662b5a64b..ad7c7a8cd 100644 --- a/android/gatt.c +++ b/android/gatt.c @@ -894,11 +894,17 @@ static void disconnect_notify_by_device(void *data, void *user_data) if (dev != conn->device || !conn->app) return; - if (dev->state == DEVICE_CONNECTED) + switch (dev->state) { + case DEVICE_CONNECTED: send_app_disconnect_notify(conn, GATT_SUCCESS); - else if (dev->state == DEVICE_CONNECT_INIT || - dev->state == DEVICE_CONNECT_READY) + break; + case DEVICE_CONNECT_INIT: + case DEVICE_CONNECT_READY: send_app_connect_notify(conn, GATT_FAILURE); + break; + case DEVICE_DISCONNECTED: + break; + } } static void destroy_connection(void *data) -- 2.47.3