Henri Verbeet wrote:
2009/9/5 Stefan Dösinger <stefan(a)codeweavers.com>:
+ /* All GL_ATI_meminfo enums return 4 ints, even the (undocumented) + * GL_TOTAL_PHYSICAL_MEMORY_ATI one, which returns {mem, 0, 0, 0} */ + GLint mem[4]; + /* Returns the vidmem in KB */ + glGetIntegerv(GL_TOTAL_PHYSICAL_MEMORY_ATI, mem); + checkGLcall("glGetIntegerv(GL_TOTAL_PHYSICAL_MEMORY_ATI, mem)"); + + /* The first driver version that supports ATI_meminfo doesn't implement + * GL_TOTAL_PHYSICAL_MEMORY_ATI - catch this situation as adviced by AMD + */ + if(mem[0] < 65536)
If the implementation doesn't recognise "GL_TOTAL_PHYSICAL_MEMORY_ATI" you're testing uninitialized memory here. I hope AMD didn't actually advice you to code it like that.
Stefan: There is or was a lurker here from AMD. Maybe you could forward your patches to them to get the extension fixed? James McKenzie