From: Eric Engestrom <eric@engestrom.ch>
Date: Tue, 18 Aug 2020 09:45:34 +0000 (+0200)
Subject: egl: drop an indentation level in _eglFindDisplay() by replacing break/if with a... 
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=8b2fc1d1b5a822692321af1a2a01dddbc9cff356;p=mesa.git

egl: drop an indentation level in _eglFindDisplay() by replacing break/if with a goto

Signed-off-by: Eric Engestrom <eric@engestrom.ch>
Reviewed-by: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6361>
---

diff --git a/src/egl/main/egldisplay.c b/src/egl/main/egldisplay.c
index d882a8f617c..3df75ccd46c 100644
--- a/src/egl/main/egldisplay.c
+++ b/src/egl/main/egldisplay.c
@@ -255,31 +255,30 @@ _eglFindDisplay(_EGLPlatformType plat, void *plat_dpy,
    for (disp = _eglGlobal.DisplayList; disp; disp = disp->Next) {
       if (disp->Platform == plat && disp->PlatformDisplay == plat_dpy &&
           _eglSameAttribs(disp->Options.Attribs, attrib_list))
-         break;
+         goto out;
    }
 
    /* create a new display */
-   if (!disp) {
-      disp = calloc(1, sizeof(_EGLDisplay));
-      if (disp) {
-         mtx_init(&disp->Mutex, mtx_plain);
-         disp->Platform = plat;
-         disp->PlatformDisplay = plat_dpy;
-         num_attribs = _eglNumAttribs(attrib_list);
-         if (num_attribs) {
-            disp->Options.Attribs = calloc(num_attribs, sizeof(EGLAttrib));
-            if (!disp->Options.Attribs) {
-               free(disp);
-               disp = NULL;
-               goto out;
-            }
-            memcpy(disp->Options.Attribs, attrib_list,
-                   num_attribs * sizeof(EGLAttrib));
+   assert(!disp);
+   disp = calloc(1, sizeof(_EGLDisplay));
+   if (disp) {
+      mtx_init(&disp->Mutex, mtx_plain);
+      disp->Platform = plat;
+      disp->PlatformDisplay = plat_dpy;
+      num_attribs = _eglNumAttribs(attrib_list);
+      if (num_attribs) {
+         disp->Options.Attribs = calloc(num_attribs, sizeof(EGLAttrib));
+         if (!disp->Options.Attribs) {
+            free(disp);
+            disp = NULL;
+            goto out;
          }
-         /* add to the display list */
-         disp->Next = _eglGlobal.DisplayList;
-         _eglGlobal.DisplayList = disp;
+         memcpy(disp->Options.Attribs, attrib_list,
+                num_attribs * sizeof(EGLAttrib));
       }
+      /* add to the display list */
+      disp->Next = _eglGlobal.DisplayList;
+      _eglGlobal.DisplayList = disp;
    }
 
 out: