[Piglit] Update some docstrings v2

[Dylan Baker]

On Friday, March 14, 2014 06:11:55 PM Ilia Mirkin wrote:
> On Mon, Mar 10, 2014 at 9:25 PM, Dylan Baker <baker.dylan.c at gmail.com> 
wrote:
> > It really isn't fair to still call this series about updating
> > docstrings, as patch 3 is a fairly substantial change. I took Ilia's
> > advice on that patch, and rather than using subclasses created constant
> > instances of the Status class to represent different statuses. This
> > should be cleaner and easier to understand than the current
> > implementation.
> 
> Thanks for considering my comment :) I looked through the series and
> didn't see anything objectionable, but I'm not too in tune with the
> various tests.
> 
> One thing that I was going to fix, but it would cause you merge
> conflicts, is that right now dmesg-fail is considered to be "better"
> than fail (when computing fixes/regressions). Similarly for dmesg-warn
> vs warn. Mind flipping them? Should be a separate patch at the end of
> the series. Or I can do it when you push.
> 
> Thanks,
> 
>   -ilia

I'll throw a patch at the end of the series, do you want me to send out a v3 
or just tack them on?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20140314/9fbba7f6/attachment.sig>