ecore-drm: refactor _ecore_drm_output_backlight_init() to use fewer gotos
authorMike Blumenkrantz <zmike@osg.samsung.com>
Thu, 23 Apr 2015 17:21:32 +0000 (13:21 -0400)
committerMike Blumenkrantz <zmike@osg.samsung.com>
Thu, 23 Apr 2015 17:21:32 +0000 (13:21 -0400)
src/lib/ecore_drm/ecore_drm_output.c

index 9699e98..dbcd088 100644 (file)
@@ -352,27 +352,24 @@ _ecore_drm_output_backlight_init(Ecore_Drm_Output *output, uint32_t conn_type)
         else if (!strcmp(devtype, "firmware"))
           type = ECORE_DRM_BACKLIGHT_FIRMWARE;
 
-        if ((conn_type != DRM_MODE_CONNECTOR_LVDS) && 
-            (conn_type != DRM_MODE_CONNECTOR_eDP))
-          {
-             if (type != ECORE_DRM_BACKLIGHT_RAW) goto cont;
-          }
+        if ((conn_type == DRM_MODE_CONNECTOR_LVDS) ||
+            (conn_type == DRM_MODE_CONNECTOR_eDP) ||
+            (type == ECORE_DRM_BACKLIGHT_RAW))
+          found = EINA_TRUE;
 
-        found = EINA_TRUE;
-cont:
         eina_stringshare_del(devtype);
         if (found) break;
      }
 
-   if (!found) goto out;
-
-   if ((backlight = calloc(1, sizeof(Ecore_Drm_Backlight))))
+   if (found)
      {
-        backlight->type = type;
-        backlight->device = eina_stringshare_add(device);
+        if ((backlight = calloc(1, sizeof(Ecore_Drm_Backlight))))
+          {
+             backlight->type = type;
+             backlight->device = eina_stringshare_add(device);
+          }
      }
 
-out:
    EINA_LIST_FREE(devs, device)
      eina_stringshare_del(device);