[Piglit] [PATCH] Create an initial to-do list

Brian Paul brianp at vmware.com
Thu Sep 25 08:02:55 PDT 2014


I'm sure other people have items to add...
---
 TODO |   28 ++++++++++++++++++++++++++++
 1 file changed, 28 insertions(+)
 create mode 100644 TODO

diff --git a/TODO b/TODO
new file mode 100644
index 0000000..fb5c405
--- /dev/null
+++ b/TODO
@@ -0,0 +1,28 @@
+
+Piglit To-Do items
+
+
+Port remaining Glean tests to Piglit (discard Glean tests which are
+already covered by Piglit).
+See https://bugs.freedesktop.org/show_bug.cgi?id=65676 for more details.
+
+
+Move/rename tests in the bugs/ directory to proper places under tests/spec/
+
+
+Replace calls such as assert(glGetError() == 0) with piglit_check_gl_error().
+
+
+Quite a few tests have GL enum names encoded in tables.  For example, in
+tests/texturing/texture-rg.c:
+
+static const struct format_info IntFormats[] = {
+   { "GL_R8", GL_R8, GL_RED, GL_FALSE, GL_FALSE, GL_FALSE },
+   { "GL_R16", GL_R16, GL_RED, GL_FALSE, GL_FALSE, GL_FALSE },
+   { "GL_RG8", GL_RG8, GL_RG, GL_FALSE, GL_FALSE, GL_FALSE },
+[...]
+
+There's no reason to do this.  The program could use piglit_get_gl_enum_name()
+instead to convert GL enums to string names.
+
+
-- 
1.7.10.4



More information about the Piglit mailing list