[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