Mesa (master): i965: Remove brwIsProgramNative

Ian Romanick idr at kemper.freedesktop.org
Sat Oct 25 02:55:02 UTC 2014


Module: Mesa
Branch: master
Commit: 4939c2eced8834a448fea894785ba5bfdf1076fd
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=4939c2eced8834a448fea894785ba5bfdf1076fd

Author: Ian Romanick <ian.d.romanick at intel.com>
Date:   Mon Oct 20 15:40:44 2014 -0700

i965: Remove brwIsProgramNative

Originally I just fixed some unused parameter warnings in this
function.  However, Ken pointed out:

    "You could instead remove this driver hook.  If the dd pointer is
    NULL, arbprogram.c will return true.  I think I'd prefer that."

Way, way back in time, I think _mesa_GetProgramivARB had the opposite
behavior.  Given that it works the way it now works, I also prefer
removing the driver hook.

Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>

---

 src/mesa/drivers/dri/i965/brw_program.c |    9 ---------
 1 file changed, 9 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_program.c b/src/mesa/drivers/dri/i965/brw_program.c
index e56ece0..ed117dd 100644
--- a/src/mesa/drivers/dri/i965/brw_program.c
+++ b/src/mesa/drivers/dri/i965/brw_program.c
@@ -120,14 +120,6 @@ static void brwDeleteProgram( struct gl_context *ctx,
 
 
 static GLboolean
-brwIsProgramNative(struct gl_context *ctx,
-		   GLenum target,
-		   struct gl_program *prog)
-{
-   return true;
-}
-
-static GLboolean
 brwProgramStringNotify(struct gl_context *ctx,
 		       GLenum target,
 		       struct gl_program *prog)
@@ -233,7 +225,6 @@ void brwInitFragProgFuncs( struct dd_function_table *functions )
 
    functions->NewProgram = brwNewProgram;
    functions->DeleteProgram = brwDeleteProgram;
-   functions->IsProgramNative = brwIsProgramNative;
    functions->ProgramStringNotify = brwProgramStringNotify;
 
    functions->NewShader = brw_new_shader;




More information about the mesa-commit mailing list