From 96bf6abb6615a9175f56a85322dd98160291274d Mon Sep 17 00:00:00 2001 From: Keith Whitwell Date: Thu, 24 Nov 2005 10:59:10 +0000 Subject: Use _mesa_exec_free for fp->func. --- src/mesa/tnl/t_vertex.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) (limited to 'src/mesa/tnl/t_vertex.c') diff --git a/src/mesa/tnl/t_vertex.c b/src/mesa/tnl/t_vertex.c index cf67dfc179..ca3aad1404 100644 --- a/src/mesa/tnl/t_vertex.c +++ b/src/mesa/tnl/t_vertex.c @@ -494,14 +494,14 @@ void _tnl_free_vertices( GLcontext *ctx ) for (fp = vtx->fastpath ; fp ; fp = tmp) { tmp = fp->next; FREE(fp->attr); - /* XXX Can we be sure that fp->func points to memory that was - * allocated with _mesa_exec_malloc()? I think it's possible that - * the pointer can point to an ordinary C function. - * Just disable this for the time being. See bug 5131. + + /* KW: At the moment, fp->func is constrained to be allocated by + * _mesa_exec_alloc(), as the hardwired fastpaths in + * t_vertex_generic.c are handled specially. It would be nice + * to unify them, but this probably won't change until this + * module gets another overhaul. */ -#if 0 _mesa_exec_free((void *) fp->func); -#endif FREE(fp); } -- cgit v1.2.3