[Piglit] [PATCH 13/26] patterns.py: PEP8 compliance

Dylan Baker baker.dylan.c at gmail.com
Thu Jul 11 11:44:07 PDT 2013


I think you're right. But all I've changed here is the indenting not the
content, and I don't think it really belongs in this patchset.

If I make another patch with that change can I put your reviewed-by on it?


On Wed, Jul 10, 2013 at 5:33 PM, Chris Forbes <chrisf at ijw.co.nz> wrote:

> +    NOTE: deleting a singleton instance (i.e. Singleton::delInstance)
> does not
> +    guarantee that something else is currently using it. To reduce this
> risk, a
>
> Shouldn't this read '... does not guarantee that nothing else is
> currently using it'?
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20130711/81cf91f6/attachment.html>


More information about the Piglit mailing list