From cbc859f17d2a9e1204e6b020df8f328e3bcd2c9d Mon Sep 17 00:00:00 2001
From: =?utf8?q?Jos=C3=A9=20Fonseca?= <jose.r.fonseca@gmail.com>
Date: Thu, 5 Apr 2012 19:55:53 +0100
Subject: [PATCH] Handle glMapObjectBufferATI/glUnmapObjectBufferATI.

---
 glretrace.py   | 11 +++++++++--
 specs/glapi.py |  2 +-
 2 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/glretrace.py b/glretrace.py
index bb88288..97a6ecc 100644
--- a/glretrace.py
+++ b/glretrace.py
@@ -159,7 +159,8 @@ class GlRetracer(Retracer):
         'glMapBufferOES',
         'glMapBufferRange',
         'glMapNamedBufferEXT',
-        'glMapNamedBufferRangeEXT'
+        'glMapNamedBufferRangeEXT',
+        'glMapObjectBufferATI',
     ])
 
     unmap_function_names = set([
@@ -167,6 +168,7 @@ class GlRetracer(Retracer):
         'glUnmapBufferARB',
         'glUnmapBufferOES',
         'glUnmapNamedBufferEXT',
+        'glUnmapObjectBufferATI',
     ])
 
     def retraceFunctionBody(self, function):
@@ -266,6 +268,9 @@ class GlRetracer(Retracer):
                 print r'            glGetBufferPointervOES(target, GL_BUFFER_MAP_POINTER_OES, &ptr);'
             elif function.name == 'glUnmapNamedBufferEXT':
                 print r'            glGetNamedBufferPointervEXT(buffer, GL_BUFFER_MAP_POINTER, &ptr);'
+            elif function.name == 'glUnmapObjectBufferATI':
+                # TODO
+                pass
             else:
                 assert False
             print r'        if (ptr) {'
@@ -338,7 +343,7 @@ class GlRetracer(Retracer):
                 print r'        if (!__result) {'
                 print r'             retrace::warning(call) << "failed to map buffer\n";'
                 print r'        }'
-            if function.name in self.unmap_function_names:
+            if function.name in self.unmap_function_names and function.type is not stdapi.Void:
                 print r'        if (!__result) {'
                 print r'             retrace::warning(call) << "failed to unmap buffer\n";'
                 print r'        }'
@@ -368,6 +373,8 @@ class GlRetracer(Retracer):
                     print r'    glGetBufferParameterivARB(target, GL_BUFFER_SIZE_ARB, &length);'
                 elif function.name == 'glMapNamedBufferEXT':
                     print r'    glGetNamedBufferParameterivEXT(buffer, GL_BUFFER_SIZE, &length);'
+                elif function.name == 'glMapObjectBufferATI':
+                    print r'    glGetObjectBufferivATI(buffer, GL_OBJECT_BUFFER_SIZE_ATI, &length);'
                 else:
                     assert False
 
diff --git a/specs/glapi.py b/specs/glapi.py
index 886da1c..ee86b39 100644
--- a/specs/glapi.py
+++ b/specs/glapi.py
@@ -2142,7 +2142,7 @@ glapi.addFunctions([
     GlFunction(Void, "glPrimitiveRestartIndexNV", [(GLuint, "index")]),
 
     # GL_ATI_map_object_buffer
-    GlFunction(GLpointer, "glMapObjectBufferATI", [(GLbuffer, "buffer")]),
+    GlFunction(GLmap, "glMapObjectBufferATI", [(GLbuffer, "buffer")]),
     GlFunction(Void, "glUnmapObjectBufferATI", [(GLbuffer, "buffer")]),
 
     # GL_ATI_separate_stencil
-- 
2.43.0