[Mesa-dev] [PATCH 01/11] meta: handle subimages in _mesa_meta_setup_texture_coords()

Brian Paul brianp at vmware.com
Wed Dec 17 11:40:41 PST 2014


On 12/16/2014 12:56 PM, Laura Ekstrand wrote:
>
>
> On Sat, Dec 13, 2014 at 6:42 AM, Brian Paul <brianp at vmware.com
> <mailto:brianp at vmware.com>> wrote:
>
>     In preparation for getting texture sub images.
>     ---
>       src/mesa/drivers/common/meta.c                 | 88
>     +++++++++++++++++---------
>       src/mesa/drivers/common/meta.h                 |  4 ++
>       src/mesa/drivers/common/meta_generate_mipmap.c |  4 +-
>       3 files changed, 64 insertions(+), 32 deletions(-)
>
>     diff --git a/src/mesa/drivers/common/meta.c
>     b/src/mesa/drivers/common/meta.c
>     index 87532c1..a84e512 100644
>     --- a/src/mesa/drivers/common/meta.c
>     +++ b/src/mesa/drivers/common/meta.c
>     @@ -2450,30 +2450,53 @@ _mesa_meta_Bitmap(struct gl_context *ctx,
>
>       /**
>        * Compute the texture coordinates for the four vertices of a quad for
>     - * drawing a 2D texture image or slice of a cube/3D texture.
>     + * drawing a 2D texture image or slice of a cube/3D texture.  The
>     offset
>     + * and width, height specify a sub-region of the 2D image.
>     + *
>        * \param faceTarget  GL_TEXTURE_1D/2D/3D or cube face name
>        * \param slice  slice of a 1D/2D array texture or 3D texture
>     - * \param width  width of the texture image
>     - * \param height  height of the texture image
>     + * \param xoffset  X position of sub texture
>     + * \param yoffset  Y position of sub texture
>     + * \param width  width of the sub texture image
>     + * \param height  height of the sub texture image
>     + * \param total_width  total width of the texture image
>     + * \param total_height  total height of the texture image
>     + * \param total_depth  total depth of the texture image
>        * \param coords0/1/2/3  returns the computed texcoords
>        */
>
>
> I'm confused.  Here you take the args width, height, then total_width,
> total_height, and total_depth.  But in the definition of
> _mesa_meta_setup_texture_coords in meta.h, you have width, height,
> depth, total_width, and total_height.

Typo.  I've fixed the meta.h declaration.

-Brian




More information about the mesa-dev mailing list