On 21 May 2012 11:08, Pauli Nieminen <span dir="ltr">&lt;<a href="mailto:pauli.nieminen@linux.intel.com" target="_blank">pauli.nieminen@linux.intel.com</a>&gt;</span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Enums need to have complete name to allow mixing EGL and GL enums. Add<br>
GL_ prefix to spec enums.<br></blockquote><div><br>Can you update the documentation at the top of parse_glspec.py to reflect this change?  This line needs to change:<br><br>#     &lt;enum name, without &quot;GL_&quot; prefix&gt;: {<br>
 </div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Signed-off-by: Pauli Nieminen &lt;<a href="mailto:pauli.nieminen@linux.intel.com">pauli.nieminen@linux.intel.com</a>&gt;<br>
---<br>
 glapi/parse_glspec.py      |    4 +++-<br>
 tests/util/gen_dispatch.py |    2 +-<br>
 2 files changed, 4 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/glapi/parse_glspec.py b/glapi/parse_glspec.py<br>
index 9d7d523..16842a6 100644<br>
--- a/glapi/parse_glspec.py<br>
+++ b/glapi/parse_glspec.py<br>
@@ -461,8 +461,10 @@ class Api(object):<br>
     #                             &#39;value_str&#39;: &quot;0xFFFFFFFF&quot; }<br>
     def parse_enum(self, m):<br>
        name, value = m.groups()<br>
+       if not name.startswith(&#39;GL_&#39;):<br>
+           name = &#39;GL_&#39; + name<br>
        if value.startswith(&#39;GL_&#39;):<br>
-           value_rhs = value[3:]<br>
+           value_rhs = value<br>
            value_int = self.enums[value_rhs][&#39;value_int&#39;]<br>
        else:<br>
            value_int = decode_enum_value(value)<br>
diff --git a/tests/util/gen_dispatch.py b/tests/util/gen_dispatch.py<br>
index 345260a..63649bc 100644<br>
--- a/tests/util/gen_dispatch.py<br>
+++ b/tests/util/gen_dispatch.py<br>
@@ -620,7 +620,7 @@ def generate_code(api):<br>
<br>
     # Emit enum #defines<br>
     for name, value in api.compute_unique_enums():<br>
-       h_contents.append(&#39;#define GL_{0} {1}\n&#39;.format(name, value))<br>
+       h_contents.append(&#39;#define {0} {1}\n&#39;.format(name, value))<br>
<br>
     # Emit extension #defines<br>
     h_contents.append(&#39;\n&#39;)<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.7.5.4<br>
<br>
_______________________________________________<br>
Piglit mailing list<br>
<a href="mailto:Piglit@lists.freedesktop.org">Piglit@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
</font></span></blockquote></div><br>