[Piglit] Question on commit 4259ce: all.tests: Remove glslparsertest CorrectPreprocess11.frag test

[Vinson Lee]

> -----Original Message-----
> From: Ian Romanick [mailto:idr at freedesktop.org]
> Sent: Monday, January 10, 2011 8:37 AM
> To: chad at chad-versace.us
> Cc: Vinson Lee; piglit at lists.freedesktop.org
> Subject: Re: [Piglit] Question on commit 4259ce: all.tests: Remove
> glslparsertest CorrectPreprocess11.frag test
> 
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> On 01/09/2011 03:14 PM, Chad Versace wrote:
> > Vinson, I have a question on commit 4259ce. (For reference, the commit
> > is reproduced below.) Could you please explain what is wrong with the
> > test CorrectPreprocess11.frag?
> >
> > Also, if the test is indeed incorrect, I believe the proper way to
> > remove it would have been to delete the test file itself.
> 
> All of the tests in the shaders/ directory came from the old 3dlabs GLSL
> tests.  We've had an implicit goal of not modifying those tests.  This
> is why all of our new tests were added in the glsl2/ directory.  It is
> worth noting that many of these tests were imported into the OpenGL ES
> 2.0 test suite.  However, CorrectPreprocess11.frag is not one of them.
> 
> The problem with CorrectPreprocess11.frag is that it uses incorrect
> #ifdef syntax.  Line 43 (shown below) is wrong.
> 
> #ifdef pine please raise a warning
> 
> The test also contains an erroneous macro redefinition at line 56.
> 
> This test was also not included in the test list prior to 71bb53336
> (glslparsertest: Replace manual listing of tests with auto-discovery).
> Since this test doesn't include a [config] section, it shouldn't be
> included now either.  In my test results it shows up a "skip".

The test also reported as a "skip" in my results, but reported as a "fail" in the valgrind section.

Other skipped tests are reported as pass in the valgrind section, but CorrectPreprocess11.frag is not. I removed CorrectPreprocess11.frag to avoid this failure from appearing in the test results.


> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.10 (GNU/Linux)
> Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
> 
> iEYEARECAAYFAk0rNcEACgkQX1gOwKyEAw8xiwCdGc0R/r7aVBsIAkK8OEiM273o
> +/IAoIxJ4ieYknkSROwFkNC3yJKjt5kd
> =FDYF
> -----END PGP SIGNATURE-----