[Mesa-dev] [PATCH 22/24] radeon: Remove early dereference of src/dst width in glCopyTexSubImage.
Eric Anholt
eric at anholt.net
Fri Oct 28 12:50:12 PDT 2011
There doesn't appear to be any particular reason for this -- it's not
like the width is changing between the deref and the use.
---
src/mesa/drivers/dri/radeon/radeon_tex_copy.c | 8 ++------
1 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/src/mesa/drivers/dri/radeon/radeon_tex_copy.c b/src/mesa/drivers/dri/radeon/radeon_tex_copy.c
index 47b4180..bc9015e 100644
--- a/src/mesa/drivers/dri/radeon/radeon_tex_copy.c
+++ b/src/mesa/drivers/dri/radeon/radeon_tex_copy.c
@@ -51,8 +51,6 @@ do_copy_texsubimage(struct gl_context *ctx,
unsigned dst_bpp;
gl_format src_mesaformat;
gl_format dst_mesaformat;
- unsigned src_width;
- unsigned dst_width;
unsigned flip_y;
if (!radeon->vtbl.blit) {
@@ -101,8 +99,6 @@ do_copy_texsubimage(struct gl_context *ctx,
src_mesaformat = rrb->base.Format;
dst_mesaformat = timg->base.Base.TexFormat;
- src_width = rrb->base.Width;
- dst_width = timg->base.Base.Width;
src_bpp = _mesa_get_format_bytes(src_mesaformat);
dst_bpp = _mesa_get_format_bytes(dst_mesaformat);
if (!radeon->vtbl.check_blit(dst_mesaformat)) {
@@ -133,10 +129,10 @@ do_copy_texsubimage(struct gl_context *ctx,
/* blit from src buffer to texture */
return radeon->vtbl.blit(ctx, rrb->bo, src_offset, src_mesaformat, rrb->pitch/rrb->cpp,
- src_width, rrb->base.Height, x, y,
+ rrb->base.Width, rrb->base.Height, x, y,
timg->mt->bo, dst_offset, dst_mesaformat,
timg->mt->levels[level].rowstride / dst_bpp,
- dst_width, timg->base.Base.Height,
+ timg->base.Base.Width, timg->base.Base.Height,
dstx, dsty, width, height, flip_y);
}
--
1.7.7
More information about the mesa-dev
mailing list