draw: Clean up single-use goto statements.
authorVinson Lee <vlee@freedesktop.org>
Sat, 31 Oct 2020 02:29:39 +0000 (19:29 -0700)
committerMarge Bot <eric+marge@anholt.net>
Wed, 11 Nov 2020 18:36:37 +0000 (18:36 +0000)
Suggested-by: Ian Romanick <ian.d.romanick@intel.com>
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7389>

src/gallium/auxiliary/draw/draw_pipe_aaline.c

index de6f970..71a84f2 100644 (file)
@@ -308,16 +308,11 @@ generate_aaline_fs(struct aaline_stage *aaline)
 #endif
 
    aaline->fs->aaline_fs = aaline->driver_create_fs_state(pipe, &aaline_fs);
-   if (aaline->fs->aaline_fs == NULL)
-      goto fail;
-
-   aaline->fs->generic_attrib = transform.maxGeneric + 1;
-   FREE((void *)aaline_fs.tokens);
-   return TRUE;
+   if (aaline->fs->aaline_fs != NULL)
+      aaline->fs->generic_attrib = transform.maxGeneric + 1;
 
-fail:
    FREE((void *)aaline_fs.tokens);
-   return FALSE;
+   return aaline->fs->aaline_fs != NULL;
 }
 
 static boolean
@@ -336,13 +331,12 @@ generate_aaline_fs_nir(struct aaline_stage *aaline)
    nir_lower_aaline_fs(aaline_fs.ir.nir, &aaline->fs->generic_attrib);
    aaline->fs->aaline_fs = aaline->driver_create_fs_state(pipe, &aaline_fs);
    if (aaline->fs->aaline_fs == NULL)
-      goto fail;
+      return FALSE;
 
    return TRUE;
-
-fail:
-#endif
+#else
    return FALSE;
+#endif
 }
 
 /**
@@ -606,15 +600,12 @@ draw_aaline_stage(struct draw_context *draw)
    aaline->stage.reset_stipple_counter = aaline_reset_stipple_counter;
    aaline->stage.destroy = aaline_destroy;
 
-   if (!draw_alloc_temp_verts(&aaline->stage, 8))
-      goto fail;
+   if (!draw_alloc_temp_verts(&aaline->stage, 8)) {
+      aaline->stage.destroy(&aaline->stage);
+      return NULL;
+   }
 
    return aaline;
-
- fail:
-   aaline->stage.destroy(&aaline->stage);
-
-   return NULL;
 }