<div dir="ltr"><div>Patches 16 to 18: these look reasonable.<br></div>Patches 1 to 18: did these get tested on windows ( and mac osx ) by any chance? </div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 28, 2014 at 5:35 PM, Dylan Baker <span dir="ltr"><<a href="mailto:baker.dylan.c@gmail.com" target="_blank">baker.dylan.c@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This allows more state in glean to be shared rather than storing the<br>
same list in memory once for each glean test.<br>
<br>
Signed-off-by: Dylan Baker <<a href="mailto:baker.dylan.c@gmail.com">baker.dylan.c@gmail.com</a>><br>
---<br>
 framework/exectest.py  |  8 +++++---<br>
 framework/gleantest.py | 12 +++++++-----<br>
 2 files changed, 12 insertions(+), 8 deletions(-)<br>
<br>
diff --git a/framework/exectest.py b/framework/exectest.py<br>
index d1441c9..0787442 100644<br>
--- a/framework/exectest.py<br>
+++ b/framework/exectest.py<br>
@@ -69,10 +69,13 @@ class Test(object):<br>
     run_concurrent -- If True the test is thread safe. Default: False<br>
<br>
     """<br>
-    OPTS = Options()<br>
     __slots__ = ['run_concurrent', 'env', 'result', 'cwd', '_command',<br>
                  '_test_hook_execute_run']<br>
<br>
+    OPTS = Options()<br>
+    _VALGRIND_CMD = ['valgrind', '--quiet', '--error-exitcode=1',<br>
+                     '--tool=memcheck']<br>
+<br>
     def __init__(self, command, run_concurrent=False):<br>
         self._command = None<br>
         self.run_concurrent = run_concurrent<br>
@@ -130,8 +133,7 @@ class Test(object):<br>
     def command(self):<br>
         assert self._command<br>
         if self.OPTS.valgrind:<br>
-            return ['valgrind', '--quiet', '--error-exitcode=1',<br>
-                    '--tool=memcheck'] + self._command<br>
+            return self._VALGRIND_CMD + self._command<br>
         return self._command<br>
<br>
     @command.setter<br>
diff --git a/framework/gleantest.py b/framework/gleantest.py<br>
index 1af1897..015a145 100644<br>
--- a/framework/gleantest.py<br>
+++ b/framework/gleantest.py<br>
@@ -36,16 +36,18 @@ class GleanTest(Test):<br>
<br>
     """<br>
     GLOBAL_PARAMS = []<br>
-    _EXECUTABLE = os.path.join(TEST_BIN_DIR, "glean")<br>
+    _EXECUTABLE = [os.path.join(TEST_BIN_DIR, "glean"),<br>
+                   "-o", "-v", "-v", "-v", "-t"]<br>
<br>
     def __init__(self, name, **kwargs):<br>
-        super(GleanTest, self).__init__(<br>
-            [self._EXECUTABLE, "-o", "-v", "-v", "-v", "-t", "+" + name],<br>
-            **kwargs)<br>
+        super(GleanTest, self).__init__(['+' + name], **kwargs)<br>
<br>
     @Test.command.getter<br>
     def command(self):<br>
-        return super(GleanTest, self).command + self.GLOBAL_PARAMS<br>
+        if self.OPTS.valgrind:<br>
+            return self._VALGRIND_CMD + self._EXECUTABLE + \<br>
+                self._command + self.GLOBAL_PARAMS<br>
+        return self._EXECUTABLE + self._command + self.GLOBAL_PARAMS<br>
<br>
     def interpret_result(self):<br>
         if self.result['returncode'] != 0 or 'FAIL' in self.result['out']:<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.0.2<br>
<br>
_______________________________________________<br>
Piglit mailing list<br>
<a href="mailto:Piglit@lists.freedesktop.org">Piglit@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
</font></span></blockquote></div><br></div>