On 05/30/2012 03:19 PM, Eric Anholt wrote:
On Tue, 29 May 2012 15:51:47 -0700, "Ian Romanick"<i...@freedesktop.org>  wrote:
+TEST(GetProcAddress, ABINameByOffset)
+{
+   /* 408 functions have had their locations in the dispatch table set since
+    * the danw of time.  Verify that all of these functions are at the correct
              ^dawn

D'oh!

(also copy and pasted)

I don't like the "as big as at some git sha1" test given that you
immediately go on to show that the test is invalid and just hack around
it.  That sounds to me like "this test should be removed".  Maybe just
check that it's bigger than the fixed ABI?

The intention is for the test to catch changes that accidentally cause the dispatch table to shrink. The dispatch table pretty much only ever grows. The most recent change to gl_API.xml that caused the dispatch table to shrink was a5f220607 in 2006. I'd be surprised if there were more changes after this series that shrink the dispatch table before another 6 years. :)

Anything else that causes a shrink is a bug. I want to have some kind of test for that, but it doesn't necessarily have to be this test.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to