From d3c604e12e1e2ef1e562f51e529ba18bae2d9af1 Mon Sep 17 00:00:00 2001 From: Mathias Fröhlich Date: Fri, 16 Mar 2018 06:34:35 +0100 Subject: mesa: Update VAO internal state when setting the _DrawVAO. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Update the VAO internal state on Array._DrawVAO instead of Array.VAO. Also the VAO internal state update gets triggered now by a change of Array._DrawVAO instead of the _NEW_ARRAY state flag. Also no driver looks at any VAO's NewArrays value from within the Driver.UpdateState callback. So it should be safe to move this update into the _mesa_set_draw_vao method. Reviewed-by: Brian Paul Signed-off-by: Mathias Fröhlich --- src/mesa/main/arrayobj.c | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'src/mesa/main/arrayobj.c') diff --git a/src/mesa/main/arrayobj.c b/src/mesa/main/arrayobj.c index 7cb98337198..0d2f7a918ac 100644 --- a/src/mesa/main/arrayobj.c +++ b/src/mesa/main/arrayobj.c @@ -461,6 +461,12 @@ _mesa_update_vao_derived_arrays(struct gl_context *ctx, { /* Make sure we do not run into problems with shared objects */ assert(!vao->SharedAndImmutable || vao->NewArrays == 0); + + /* + * Stay tuned, the next series scans for duplicate bindings in this + * function. So that drivers can easily know the minimum unique set + * of bindings. + */ } -- cgit v1.2.3