[Mesa-dev] [PATCH 1/5] st/mesa: simplify some code in get_texture_format_swizzle()

Edward O'Callaghan funfunctor at folklore1984.net
Sat Oct 1 05:07:49 UTC 2016


This series is,
Reviewed-by: Edward O'Callaghan <funfunctor at folklore1984.net>

On 10/01/2016 08:53 AM, Brian Paul wrote:
> There's no need to cast to st_texture_image.  Just use gl_texture_image.
> ---
>  src/mesa/state_tracker/st_atom_texture.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/src/mesa/state_tracker/st_atom_texture.c b/src/mesa/state_tracker/st_atom_texture.c
> index efc8c90..113c0ed 100644
> --- a/src/mesa/state_tracker/st_atom_texture.c
> +++ b/src/mesa/state_tracker/st_atom_texture.c
> @@ -211,11 +211,11 @@ get_texture_format_swizzle(const struct st_context *st,
>         */
>        if (_mesa_is_gles3(st->ctx) &&
>            util_format_is_depth_or_stencil(stObj->pt->format)) {
> -         const struct st_texture_image *firstImage =
> -            st_texture_image_const(_mesa_base_tex_image(&stObj->base));
> -         if (firstImage->base.InternalFormat != GL_DEPTH_COMPONENT &&
> -             firstImage->base.InternalFormat != GL_DEPTH_STENCIL &&
> -             firstImage->base.InternalFormat != GL_STENCIL_INDEX)
> +         const struct gl_texture_image *firstImage =
> +            _mesa_base_tex_image(&stObj->base);
> +         if (firstImage->InternalFormat != GL_DEPTH_COMPONENT &&
> +             firstImage->InternalFormat != GL_DEPTH_STENCIL &&
> +             firstImage->InternalFormat != GL_STENCIL_INDEX)
>              depth_mode = GL_RED;
>        }
>        tex_swizzle = compute_texture_format_swizzle(baseFormat,
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20161001/1accbedc/attachment.sig>


More information about the mesa-dev mailing list