On 10/15/2013 05:32 PM, Ian Romanick wrote:
From: Ian Romanick <ian.d.roman...@intel.com>

Create a new function, piglit_gl_process_args, that does the
command-line parsing.  This happens after the test code between
PIGLIT_GL_TEST_CONFIG_BEGIN and PIGLIT_GL_TEST_CONFIG_END.  By having an
explicit function that does this, tests can call it inside the BEGIN/END
block.  This may be useful for tests that expect certain arguments to be
in specific positions.

Signed-off-by: Ian Romanick <ian.d.roman...@intel.com>
Cc: Chad Versace <chad.vers...@linux.intel.com>
---
  tests/texturing/shaders/texelFetch.c  | 10 ++++++----
  tests/texturing/shaders/textureSize.c | 10 ++++++----
  tests/util/piglit-framework-gl.c      | 24 ++++++++++++++----------
  tests/util/piglit-framework-gl.h      | 10 +++++++---
  4 files changed, 33 insertions(+), 21 deletions(-)


As Paul requested, explain in a comment that it's safe to call
piglit_gl_process_args() twice b/c it's idempotent, and patch 1
is

Reviewed-by: Chad Versace <chad.vers...@linux.intel.com>

_______________________________________________
Piglit mailing list
Piglit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to