Module: Mesa
Branch: master
Commit: d1211f3112c0e7c710a380ddc0887a44f461e1bb
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=d1211f3112c0e7c710a380ddc0887a44f461e1bb

Author: Emil Velikov <[email protected]>
Date:   Wed Aug  8 15:05:56 2018 +0100

egl/android: remove droid_probe_driver()

The function name is misleading - it effectively checks if
loader_get_driver_for_fd fails. Which can happen only only on strdup
error - a close to impossible scenario.

Drop the function - we call the loader API at at later stage.

Signed-off-by: Emil Velikov <[email protected]>
Reviewed-by: Tomasz Figa <[email protected]>

---

 src/egl/drivers/dri2/platform_android.c | 18 ------------------
 1 file changed, 18 deletions(-)

diff --git a/src/egl/drivers/dri2/platform_android.c 
b/src/egl/drivers/dri2/platform_android.c
index ee6c2791b1..de53376c9f 100644
--- a/src/egl/drivers/dri2/platform_android.c
+++ b/src/egl/drivers/dri2/platform_android.c
@@ -1404,19 +1404,6 @@ error:
    return false;
 }
 
-static bool
-droid_probe_driver(int fd)
-{
-   char *driver_name;
-
-   driver_name = loader_get_driver_for_fd(fd);
-   if (driver_name == NULL)
-      return false;
-
-   free(driver_name);
-   return true;
-}
-
 typedef enum {
    probe_fail = -1,
    probe_success = 0,
@@ -1442,11 +1429,6 @@ droid_probe_device(_EGLDisplay *disp, int fd, const char 
*vendor)
       goto cleanup;
    }
 
-   if (!droid_probe_driver(fd)) {
-      ret = probe_fail;
-      goto cleanup;
-   }
-
    ret = probe_success;
 
 cleanup:

_______________________________________________
mesa-commit mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-commit

Reply via email to