On 10/24/2012 02:20 PM, Jordan Justen wrote:
For newer versions of GL, we can indicate that GL requires the
function, but Mesa does not yet support it.
To indicate a function is not yet known by Mesa, you add a dash (-)
at the start of the name.
If the function is required for the context version being tested,
then the test will assert.
Per my comments in the next patch, I think this patch can be dropped.
One further comment below...
Signed-off-by: Jordan Justen <jordan.l.jus...@intel.com>
---
src/mesa/main/tests/dispatch_sanity.cpp | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/src/mesa/main/tests/dispatch_sanity.cpp
b/src/mesa/main/tests/dispatch_sanity.cpp
index 40b3d8e..d5f0467 100644
--- a/src/mesa/main/tests/dispatch_sanity.cpp
+++ b/src/mesa/main/tests/dispatch_sanity.cpp
@@ -121,6 +121,15 @@ DispatchSanity_test::validate_functions()
? function_table[i].offset
: _glapi_get_proc_offset(function_table[i].name);
+ if (function_table[i].name[0] == '-') {
+ ASSERT_EQ (false, cant_be_nop)
+ << "Function " << (function_table[i].name + 1)
+ << " is required by GL " << (ctx.Version / 10)
+ << '.' << (ctx.Version % 10)
+ << " but MESA doesn't support it.";
ASSERT_FALSE(cant_be_nop) ...
+ continue;
+ }
+
ASSERT_NE(-1, offset)
<< "Function: " << function_table[i].name;
ASSERT_EQ(offset,
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
