Bug 104908

Summary: Texture Compression Hint not converted to enum16
Product: Mesa Reporter: Baldur Karlsson <baldurk>
Component: Mesa coreAssignee: mesa-dev
Status: RESOLVED FIXED QA Contact: mesa-dev
Severity: minor    
Priority: medium    
Version: git   
Hardware: All   
OS: All   
i915 platform: i915 features:

Description Baldur Karlsson 2018-02-01 16:34:12 UTC
In commit f96a69f916a "mesa: replace GLenum with GLenum16 in common structures (v4)" it looks like Hints.TextureCompression was missed, it's still listed as an int which leads to 0x11001100 being returned instead of 0x1100 as it reads into the next value.

This diff fixes it for me, but there might be something else that needs to update:

diff --git a/src/mesa/main/get_hash_params.py b/src/mesa/main/get_hash_params.py
index e10282a156..df082af207 100644
--- a/src/mesa/main/get_hash_params.py
+++ b/src/mesa/main/get_hash_params.py
@@ -783,7 +783,7 @@ descriptor=[
 # GL_ARB_texture_compression
 # GL_EXT_compiled_vertex_array
Comment 1 Brian Paul 2018-02-01 18:31:11 UTC
The patch looks good to me.  I'll push it in a bit.

I don't see any other issues related to Hint.TextureCompression.
Unfortunately, there's no Piglit test that hits this.
Comment 2 Brian Paul 2018-02-01 19:00:31 UTC
Fixed with commit 030821a8737cd8a7be001e40dafc1879f3df7866

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.