[Mesa-dev] [PATCH 26/26] glapi: Remove GL_NV_vertex_array_range from the dispatch table

Ian Romanick idr at freedesktop.org
Tue May 29 15:51:53 PDT 2012


From: Ian Romanick <ian.d.romanick at intel.com>

There is no GLX protocol for these functions.  Open-source Linux
driver have not supported this extension for many years, and it seems
unlikely at this point that this support will return.  There's no
reason to have slots for these functions in the dispatch table.

The unit test is also updated.  The odd side effect here is the test
GetProcAddress::TableAsBigAsAt_git_b45052b now really has nothing to
do with GIT SHA1 b45052b.  This is why the new value is '978u - 2u'.
GIT SHA1 b45052b had 978 entries, and two are removed by this change.

Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
---
 src/mapi/glapi/gen/gl_API.xml        |    4 ++--
 src/mapi/glapi/tests/check_table.cpp |    7 ++++---
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/src/mapi/glapi/gen/gl_API.xml b/src/mapi/glapi/gen/gl_API.xml
index aef4d82..4bd0fc6 100644
--- a/src/mapi/glapi/gen/gl_API.xml
+++ b/src/mapi/glapi/gen/gl_API.xml
@@ -10539,11 +10539,11 @@
 </category>
 
 <category name="GL_NV_vertex_array_range" number="190">
-    <function name="FlushVertexArrayRangeNV" offset="assign">
+    <function name="FlushVertexArrayRangeNV">
         <glx ignore="true"/>
     </function>
 
-    <function name="VertexArrayRangeNV" offset="assign">
+    <function name="VertexArrayRangeNV">
         <param name="length" type="GLsizei"/>
         <param name="pointer" type="const GLvoid *"/>
         <glx ignore="true"/>
diff --git a/src/mapi/glapi/tests/check_table.cpp b/src/mapi/glapi/tests/check_table.cpp
index 0073d70..672a11d 100644
--- a/src/mapi/glapi/tests/check_table.cpp
+++ b/src/mapi/glapi/tests/check_table.cpp
@@ -91,8 +91,11 @@ TEST(GetProcAddress, TableAsBigAsAt_git_b45052b)
     *
     * 7 entries were removed when GL_NV_fence was removed from the dispatch
     * table.
+    *
+    * 2 entries were removed when GL_NV_vertex_array_range was removed from
+    * the dispatch table.
     */
-   EXPECT_GE(table_entries, 978u - 6u - 1u - 2u - 13u - 7u);
+   EXPECT_GE(table_entries, 978u - 6u - 1u - 2u - 13u - 7u - 2u);
 }
 
 TEST(GetProcAddress, QueriedDispatchSizeBigEnough)
@@ -1289,8 +1292,6 @@ const struct name_offset known_dispatch[] = {
    { "glFogCoordfEXT", _O(FogCoordfEXT) },
    { "glFogCoordfvEXT", _O(FogCoordfvEXT) },
    { "glBlendFuncSeparateEXT", _O(BlendFuncSeparateEXT) },
-   { "glFlushVertexArrayRangeNV", _O(FlushVertexArrayRangeNV) },
-   { "glVertexArrayRangeNV", _O(VertexArrayRangeNV) },
    { "glResizeBuffersMESA", _O(ResizeBuffersMESA) },
    { "glWindowPos2dMESA", _O(WindowPos2dMESA) },
    { "glWindowPos2dvMESA", _O(WindowPos2dvMESA) },
-- 
1.7.6.5



More information about the mesa-dev mailing list