[Piglit] [PATCH 0/4] Replace bash generator with python generator

Dylan Baker baker.dylan.c at gmail.com
Fri Mar 27 14:21:43 PDT 2015


On Fri, Mar 27, 2015 at 09:07:07PM +0000, Emil Velikov wrote:
> On 27 March 2015 at 19:48, Dylan Baker <baker.dylan.c at gmail.com> wrote:
> > Apparently patch 1 is almost a 1Mb, so...
> >
> > This is available at my github:
> > https://github.com/dcbaker/piglit submit/generate-variable-index-read
> >
> As usual a couple of trivial suggestions from me :-)
> 
> s/gen_variable_index_read_tests.list/variable_index_read_tests.list/

Fixed

> s/the follwing/The following/

Line removed, because of the change below.

> 
> About diffstat - the series contains some 100+ lines of copyright
> headers. So even if one ignores the generated artefacts, there is
> still some improvement in the LOC count.

I added a function to the helpers.mako for adding the copyright header,
so, that saves a few lines of code.

> 
> -Emil
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20150327/e20c70bc/attachment.sig>


More information about the Piglit mailing list