On 06/28/2017 08:13 AM, Timothy Arceri wrote:
---
  src/mesa/main/context.c | 23 +++++++----------------
  src/mesa/main/context.h |  3 ---
  2 files changed, 7 insertions(+), 19 deletions(-)

diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
index b801322..2ed4ae8 100644
--- a/src/mesa/main/context.c
+++ b/src/mesa/main/context.c
@@ -1834,20 +1834,6 @@ _mesa_record_error(struct gl_context *ctx, GLenum error)
/**
- * Flush commands and wait for completion.
- */
-void
-_mesa_finish(struct gl_context *ctx)
-{
-   FLUSH_VERTICES( ctx, 0 );
-   FLUSH_CURRENT( ctx, 0 );
-   if (ctx->Driver.Finish) {
-      ctx->Driver.Finish(ctx);
-   }
-}
-
-
-/**
   * Flush commands.
   */
  void
@@ -1863,7 +1849,7 @@ _mesa_flush(struct gl_context *ctx)
/**
- * Execute glFinish().
+ * Flush commands and wait for completion.
   *
   * Calls the #ASSERT_OUTSIDE_BEGIN_END_AND_FLUSH macro and the
   * dd_function_table::Finish driver callback, if not NULL.
@@ -1873,7 +1859,12 @@ _mesa_Finish(void)
  {
     GET_CURRENT_CONTEXT(ctx);
     ASSERT_OUTSIDE_BEGIN_END(ctx);
-   _mesa_finish(ctx);
+
+   FLUSH_VERTICES( ctx, 0 );
+   FLUSH_CURRENT( ctx, 0 );

Can you remove the extra spaces and a new line here?

With that, patch is:

Reviewed-by: Samuel Pitoiset <samuel.pitoi...@gmail.com>

+   if (ctx->Driver.Finish) {
+      ctx->Driver.Finish(ctx);
+   }
  }
diff --git a/src/mesa/main/context.h b/src/mesa/main/context.h
index ccb5463..4f75f57 100644
--- a/src/mesa/main/context.h
+++ b/src/mesa/main/context.h
@@ -162,9 +162,6 @@ _mesa_record_error( struct gl_context *ctx, GLenum error );
extern void
-_mesa_finish(struct gl_context *ctx);
-
-extern void
  _mesa_flush(struct gl_context *ctx);
extern void GLAPIENTRY

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to