<div dir="ltr">On 28 August 2013 16:49, Kenneth Graunke <span dir="ltr"><<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The main GL context's swtnl_im field is the VBO module's vbo_context<br>
structure.  Using the name "swtnl" in the name is confusing since<br>
some drivers use hardware texturing and lighting, but still rely on the<br>
VBO module for drawing.<br>
<br>
v2: Forward declare the type and use that instead of void *<br>
    (suggested by Eric Anholt)<br>
<br>
Signed-off-by: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>><br>
---<br>
 src/mesa/main/mtypes.h     | 3 ++-<br>
 src/mesa/vbo/vbo_context.c | 4 ++--<br>
 src/mesa/vbo/vbo_context.h | 2 +-<br>
 3 files changed, 5 insertions(+), 4 deletions(-)<br>
<br>
diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h<br>
index 22bb58c..7d56322 100644<br>
--- a/src/mesa/main/mtypes.h<br>
+++ b/src/mesa/main/mtypes.h<br>
@@ -80,6 +80,7 @@ struct prog_instruction;<br>
 struct gl_program_parameter_list;<br>
 struct set;<br>
 struct set_entry;<br>
+struct vbo_context;<br>
 /*@}*/<br>
<br>
<br>
@@ -3669,7 +3670,7 @@ struct gl_context<br>
    void *swrast_context;<br>
    void *swsetup_context;<br>
    void *swtnl_context;<br>
-   void *swtnl_im;<br>
+   struct vbo_context *vbo_context;<br>
    struct st_context *st;<br>
    void *aelt_context;<br>
    /*@}*/<br>
diff --git a/src/mesa/vbo/vbo_context.c b/src/mesa/vbo/vbo_context.c<br>
index b97313d..2aa5bbc 100644<br>
--- a/src/mesa/vbo/vbo_context.c<br>
+++ b/src/mesa/vbo/vbo_context.c<br>
@@ -152,7 +152,7 @@ GLboolean _vbo_CreateContext( struct gl_context *ctx )<br>
 {<br>
    struct vbo_context *vbo = CALLOC_STRUCT(vbo_context);<br>
<br>
-   ctx->swtnl_im = (void *)vbo;<br>
+   ctx->vbo_context = (void *) vbo;<br></blockquote><div><br></div><div>This cast isn't necessary either.  With both casts removed, everything but patch 4 is:<br><br></div><div>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
    /* Initialize the arrayelt helper<br>
     */<br>
@@ -224,7 +224,7 @@ void _vbo_DestroyContext( struct gl_context *ctx )<br>
       if (ctx->API == API_OPENGL_COMPAT)<br>
          vbo_save_destroy(ctx);<br>
       free(vbo);<br>
-      ctx->swtnl_im = NULL;<br>
+      ctx->vbo_context = NULL;<br>
    }<br>
 }<br>
<br>
diff --git a/src/mesa/vbo/vbo_context.h b/src/mesa/vbo/vbo_context.h<br>
index 27fae83..db47a8b 100644<br>
--- a/src/mesa/vbo/vbo_context.h<br>
+++ b/src/mesa/vbo/vbo_context.h<br>
@@ -90,7 +90,7 @@ struct vbo_context {<br>
<br>
 static inline struct vbo_context *vbo_context(struct gl_context *ctx)<br>
 {<br>
-   return (struct vbo_context *)(ctx->swtnl_im);<br>
+   return (struct vbo_context *) ctx->vbo_context;<br>
 }<br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
--<br>
1.8.3.4<br>
<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div></div>