<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Mar 1, 2017 at 12:21 PM, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wed, Mar 1, 2017 at 12:16 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> ---<br>
>  src/mesa/drivers/dri/i965/brw_<wbr>shader.cpp | 25 -------------------------<br>
>  src/mesa/drivers/dri/i965/brw_<wbr>vs.c       | 25 +++++++++++++++++++++++++<br>
>  2 files changed, 25 insertions(+), 25 deletions(-)<br>
><br>
> diff --git a/src/mesa/drivers/dri/i965/<wbr>brw_shader.cpp b/src/mesa/drivers/dri/i965/<wbr>brw_shader.cpp<br>
> index 28ed8dc..53576da 100644<br>
> --- a/src/mesa/drivers/dri/i965/<wbr>brw_shader.cpp<br>
> +++ b/src/mesa/drivers/dri/i965/<wbr>brw_shader.cpp<br>
> @@ -1137,31 +1137,6 @@ backend_shader::calculate_cfg(<wbr>)<br>
>     cfg = new(mem_ctx) cfg_t(&this->instructions);<br>
>  }<br>
><br>
> -/**<br>
> - * Decide which set of clip planes should be used when clipping via<br>
> - * gl_Position or gl_ClipVertex.<br>
> - */<br>
> -gl_clip_plane *brw_select_clip_planes(struct gl_context *ctx)<br>
> -{<br>
> -   if (ctx->_Shader->CurrentProgram[<wbr>MESA_SHADER_VERTEX]) {<br>
> -      /* There is currently a GLSL vertex shader, so clip according to GLSL<br>
> -       * rules, which means compare gl_ClipVertex (or gl_Position, if<br>
> -       * gl_ClipVertex wasn't assigned) against the eye-coordinate clip planes<br>
> -       * that were stored in EyeUserPlane at the time the clip planes were<br>
> -       * specified.<br>
> -       */<br>
> -      return ctx->Transform.EyeUserPlane;<br>
> -   } else {<br>
> -      /* Either we are using fixed function or an ARB vertex program.  In<br>
> -       * either case the clip planes are going to be compared against<br>
> -       * gl_Position (which is in clip coordinates) so we have to clip using<br>
> -       * _ClipUserPlane, which was transformed into clip coordinates by Mesa<br>
> -       * core.<br>
> -       */<br>
> -      return ctx->Transform._ClipUserPlane;<br>
> -   }<br>
> -}<br>
> -<br>
>  extern "C" const unsigned *<br>
>  brw_compile_tes(const struct brw_compiler *compiler,<br>
>                  void *log_data,<br>
> diff --git a/src/mesa/drivers/dri/i965/<wbr>brw_vs.c b/src/mesa/drivers/dri/i965/<wbr>brw_vs.c<br>
> index 3f11a76..fa54eb5 100644<br>
> --- a/src/mesa/drivers/dri/i965/<wbr>brw_vs.c<br>
> +++ b/src/mesa/drivers/dri/i965/<wbr>brw_vs.c<br>
> @@ -43,6 +43,31 @@<br>
><br>
>  #include "util/ralloc.h"<br>
><br>
> +/**<br>
> + * Decide which set of clip planes should be used when clipping via<br>
> + * gl_Position or gl_ClipVertex.<br>
> + */<br>
> +gl_clip_plane *brw_select_clip_planes(struct gl_context *ctx)<br>
<br>
</div></div>Fix the function declaration to be BSD-style while we're moving it.<br></blockquote><div><br></div><div>Will do<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Reviewed-by: Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>><br>
</blockquote></div><br></div><div class="gmail_extra">Thanks!<br></div></div>