At Wednesday 17 April 2013 11:35:52 Kenneth Graunke wrote:
> On 04/12/2013 04:39 PM, Jon Severinsson wrote:
> > [...]
>
> Two problems here:
>
> 1. This leaves the file open...you ought to close it after you're done
> reading from it. The easiest way to do this is by doing:
>
> with open(filename, 'rb') as f:
> exec(compile(f.read(), filename, 'exec'))
Oups, will do.
> 2. You missed a bunch of tests/*.tests files, which all use execfile()
> as well.
Oups, again.
> I'm not sure the best way to handle this...perhaps create a
> framework/util.py that defines execfile for Python 3? Then we'd need
> imports everywhere, though...
>
> Thoughts?
Unfortunately, this is not an option, as that would make the files be evaluated
in the context where we defined execfile, and not in the context where we
called
execfile. The only reason it works in all.test is that we only ever call the
function in the same context as we declared it :(.
All in all, I believe the best thing to do is just use the following code
snippet everywhere:
with open(«filename», 'rb') as f:
exec(compile(f.read(), f.name, 'exec'))
A bit more verbose, but works in all python version (2.6+), and we don't have
to wory about execution contexts...
/ Jon
_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit
