On Sun, 12 Jun 2011 22:28:08 -0700, Chad Versace <c...@chad-versace.us> wrote: > On Tue, 7 Jun 2011 11:47:13 -0700, Eric Anholt <e...@anholt.net> wrote: > > This is already pointing at 0 or Height - 1 and with an appropriate > > pitch, so no need to recompute those values per customization of the > > spans code. Cuts 3 out of 21kb of the compiled size. > > --- > > src/mesa/drivers/dri/intel/intel_span.c | 11 ++++------- > > 1 files changed, 4 insertions(+), 7 deletions(-) > > > > diff --git a/src/mesa/drivers/dri/intel/intel_span.c > > b/src/mesa/drivers/dri/intel/intel_span.c > > index 5290342..8978129 100644 > > --- a/src/mesa/drivers/dri/intel/intel_span.c > > +++ b/src/mesa/drivers/dri/intel/intel_span.c > > @@ -50,21 +50,18 @@ intel_set_span_functions(struct intel_context *intel, > > > > #define LOCAL_VARS \ > > struct intel_renderbuffer *irb = intel_renderbuffer(rb); > > \ > > - const GLint yScale = rb->Name ? 1 : -1; \ > > - const GLint yBias = rb->Name ? 0 : rb->Height - 1; > > \ > > int minx = 0, miny = 0; \ > > int maxx = rb->Width; \ > > int maxy = rb->Height; \ > > - int pitch = irb->region->pitch * irb->region->cpp; > > \ > > - void *buf = irb->region->buffer->virtual; > > \ > > + int pitch = rb->RowStride * irb->region->cpp; \ > > + void *buf = rb->Data; \ > > GLuint p; > > \ > > - (void) p; > > \ > > - (void)buf; (void)pitch; /* unused for non-gttmap. */ > > \ > > + (void) p; > > Since you immediately void out p, why even declare it?
It's used in half of the callers of this macro, but not the other half. The "(void)" suppresses unused variable warnings.
pgp7c5tYUPnCY.pgp
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
