[Piglit] [PATCH] Porting basic GL rendering tests from Glean to Piglit. Test replaces original glean tpaths.cpp test.

Juliet Fru julietfru at gmail.com
Wed Sep 9 05:36:29 PDT 2015


Thanks for the suggestions Emil.

On Wed, Sep 9, 2015, 11:51 AM Emil Velikov <emil.l.velikov at gmail.com> wrote:

> Hi Juliet,
>
> Just a friendly suggestion:
>
> When going through multiple revisions, you could consider tagging
> patches as PATCHv22 (git format-patch -v22 ...) and/or adding some
> revision history in the summary (see commit
> 7be0c99c75589b4810160772a79d7dc088bc8f0e).
>
> It should minimise the confusion of "which version is this/did it fix
> issue X" and hopefully ease your reviewers.
>
> -Emil
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20150909/f7fc4a9b/attachment.html>


More information about the Piglit mailing list