On Wed, Mar 1, 2017 at 12:16 PM, Jason Ekstrand <ja...@jlekstrand.net> wrote: > --- > src/mesa/drivers/dri/i965/brw_shader.cpp | 25 ------------------------- > src/mesa/drivers/dri/i965/brw_vs.c | 25 +++++++++++++++++++++++++ > 2 files changed, 25 insertions(+), 25 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_shader.cpp > b/src/mesa/drivers/dri/i965/brw_shader.cpp > index 28ed8dc..53576da 100644 > --- a/src/mesa/drivers/dri/i965/brw_shader.cpp > +++ b/src/mesa/drivers/dri/i965/brw_shader.cpp > @@ -1137,31 +1137,6 @@ backend_shader::calculate_cfg() > cfg = new(mem_ctx) cfg_t(&this->instructions); > } > > -/** > - * Decide which set of clip planes should be used when clipping via > - * gl_Position or gl_ClipVertex. > - */ > -gl_clip_plane *brw_select_clip_planes(struct gl_context *ctx) > -{ > - if (ctx->_Shader->CurrentProgram[MESA_SHADER_VERTEX]) { > - /* There is currently a GLSL vertex shader, so clip according to GLSL > - * rules, which means compare gl_ClipVertex (or gl_Position, if > - * gl_ClipVertex wasn't assigned) against the eye-coordinate clip > planes > - * that were stored in EyeUserPlane at the time the clip planes were > - * specified. > - */ > - return ctx->Transform.EyeUserPlane; > - } else { > - /* Either we are using fixed function or an ARB vertex program. In > - * either case the clip planes are going to be compared against > - * gl_Position (which is in clip coordinates) so we have to clip using > - * _ClipUserPlane, which was transformed into clip coordinates by Mesa > - * core. > - */ > - return ctx->Transform._ClipUserPlane; > - } > -} > - > extern "C" const unsigned * > brw_compile_tes(const struct brw_compiler *compiler, > void *log_data, > diff --git a/src/mesa/drivers/dri/i965/brw_vs.c > b/src/mesa/drivers/dri/i965/brw_vs.c > index 3f11a76..fa54eb5 100644 > --- a/src/mesa/drivers/dri/i965/brw_vs.c > +++ b/src/mesa/drivers/dri/i965/brw_vs.c > @@ -43,6 +43,31 @@ > > #include "util/ralloc.h" > > +/** > + * Decide which set of clip planes should be used when clipping via > + * gl_Position or gl_ClipVertex. > + */ > +gl_clip_plane *brw_select_clip_planes(struct gl_context *ctx)
Fix the function declaration to be BSD-style while we're moving it. Reviewed-by: Matt Turner <matts...@gmail.com> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev