Re: [Piglit] [PATCH 0/3] cleanups for ATI_fs tests

2018-01-03 Thread Eric Anholt
Miklós Máté  writes:

> Please do so. These are nice cleanups.
>
> MM

I finally got around to pushing these.  Once again, thanks for your work
on this!


signature.asc
Description: PGP signature
___
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit


Re: [Piglit] [PATCH 0/3] cleanups for ATI_fs tests

2017-12-08 Thread Miklós Máté

Please do so. These are nice cleanups.

MM

On 07/12/17 21:08, Eric Anholt wrote:

Here are a few little cleanups I made while reviewing the series.
Miklós, if you are OK with these, then I'll squash them into your
patches and push the whole set.

Eric Anholt (3):
   ati_fs: Wrap some comments in render tests.
   ati_fs: Use enums instead of magic numbers in the render tests.
   ati_fs: Do some column wrapping in the render tests.

  tests/spec/ati_fragment_shader/render-constants.c  | 25 +---
  tests/spec/ati_fragment_shader/render-default.c| 31 +++---
  tests/spec/ati_fragment_shader/render-fog.c| 28 ++---
  tests/spec/ati_fragment_shader/render-notexture.c  | 37 +++
  tests/spec/ati_fragment_shader/render-precedence.c | 25 +---
  tests/spec/ati_fragment_shader/render-sources.c| 71 +++---
  tests/spec/ati_fragment_shader/render-textargets.c | 32 +++---
  7 files changed, 174 insertions(+), 75 deletions(-)



___
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit