<div dir="ltr"> <div class="gmail_extra"> <div class="gmail_quote">On Fri, Jun 6, 2014 at 4:57 PM, Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com" target="_blank">anuj.phogat@gmail.com</a>> wrote: <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Function is utilized by next patch in the series. Cc: <<a href="mailto:mesa-stable@lists.freedesktop.org" target="_blank">mesa-stable@lists.freedesktop.org</a>> Signed-off-by: Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com" target="_blank">anuj.phogat@gmail.com</a>> --- src/mesa/main/glformats.c | 18 ++++++++++++++++++ src/mesa/main/glformats.h | 3 +++ 2 files changed, 21 insertions(+) diff --git a/src/mesa/main/glformats.c b/src/mesa/main/glformats.c index 05a3842..b95ecea 100644 --- a/src/mesa/main/glformats.c +++ b/src/mesa/main/glformats.c @@ -356,6 +356,24 @@ _mesa_bytes_per_vertex_attrib(GLint comps, GLenum type) } /** + * Test if the given format is an unsized color format. + */ +GLboolean +_mesa_is_enum_format_unsized(GLenum format) +{ + switch (format) { + case GL_RGBA: + case GL_RGB: + case GL_ALPHA: + case GL_LUMINANCE: + case GL_LUMINANCE_ALPHA: + return GL_TRUE; + default: + return GL_FALSE; </blockquote><div> </div><div>What about GL_RED, GL_RG, GL_INTENSITY, GL_DEPTH_COMPONENT, GL_DEPTH_STENICIL, GL_SRGB, and GL_SRGB_ALPHA? If we're going to add another metadata function, we should probably at least try to include these. Otherwise, someone is liable to make a mistake with it later. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> + } +} + +/** * Test if the given format is a SNORM (signed-normalized) format. */ GLboolean diff --git a/src/mesa/main/glformats.h b/src/mesa/main/glformats.h index d4856e5..b2ef77b 100644 --- a/src/mesa/main/glformats.h +++ b/src/mesa/main/glformats.h @@ -60,6 +60,9 @@ extern GLboolean _mesa_is_type_unsigned(GLenum type); extern GLboolean +_mesa_is_enum_format_unsized(GLenum format); + +extern GLboolean _mesa_is_enum_format_snorm(GLenum format); extern GLboolean -- 1.8.3.1 _______________________________________________ mesa-dev mailing list <a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a> <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a> </blockquote></div> </div></div>