On 12/26/2011 03:10 PM, Brian Paul wrote: > From: Brian Paul <bri...@vmware.com> > > The were always zero. When doing a sub-texture replacement we account > for the dstX/Y/Zoffsets when we map the texture image. So no need to > pass them into the texstore code anymore. > --- > src/mesa/drivers/dri/intel/intel_tex_subimage.c | 1 - > src/mesa/drivers/dri/nouveau/nouveau_texture.c | 4 +- > src/mesa/main/texcompress_fxt1.c | 18 +-- > src/mesa/main/texcompress_rgtc.c | 37 +--- > src/mesa/main/texcompress_s3tc.c | 39 +--- > src/mesa/main/texstore.c | 317 > +++++++---------------- > src/mesa/main/texstore.h | 1 - > src/mesa/state_tracker/st_cb_drawpixels.c | 1 - > src/mesa/state_tracker/st_cb_texture.c | 2 - > 9 files changed, 111 insertions(+), 309 deletions(-)
Nice cleanup! Both patches are: Reviewed-by: Kenneth Graunke <kenn...@whitecape.org> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev