[Piglit] [PATCH 6/7] Add SyncFileWriter for write/close synchronization on results file.

Chad Versace chad at chad-versace.us
Mon Feb 7 15:03:59 PST 2011


On 02/07/2011 12:04 PM, U. Artie Eoff wrote:
> +	from framework.threads import synchronized_self
> +	class SyncFileWriter:

It feels awkward for this import and class definition to exist at
function scope rather than module scope. Please move them outside the
definition of main().
-- 
Chad Versace
chad at chad-versace.us


More information about the Piglit mailing list