<div dir="ltr">On 8 April 2013 09:15, Eric Anholt <span dir="ltr"><<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<div>Paul Berry <<a href="mailto:stereotype441@gmail.com" target="_blank">stereotype441@gmail.com</a>> writes:<br>
<br>
> The vec4_visitor functions don't use any VS specific data from<br>
> vec4_visitor::vp.  So rename it to just "p" and change its type from<br>
> struct gl_vertex_program * to struct gl_program *.  This will allow<br>
> the code to be re-used for geometry shaders.<br>
<br>
</div>In many other places in the driver, "p" is a brw_compile.  I'd rather<br>
not overload that name.  In a couple other cases where we've had both<br>
the gl_shader_program and the gl_program, the shader_program becomes<br>
"shader_prog" (only about 8 instances in brw_vec4) and gl_program gets<br>
to be just "prog"<br>
</blockquote></div><br></div><div class="gmail_extra">That seems reasonable to me.  I'll break this out into two patches, one to rename struct gl_shader_program *prog to shader_prog, and another to change the type of vp and rename it to prog.<br>

<br>Note that gl_shader_program *prog appears in the backend_visitor base class, so renaming it will affect fs code too.<br></div></div>