<div dir="ltr"><div><div><div>Hello Brian, IIia,<br><br><br></div>I already commented the new piglit test for the GL rendering test. I will make recommended changes and send a new patch to the mailing list. I already ported the next test; waiting for your corrections too.<br><br></div>Thanks,<br></div>Juliet<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 8, 2015 at 4:11 PM, Brian Paul <span dir="ltr"><<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 09/08/2015 08:41 AM, Ilia Mirkin wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Tue, Sep 8, 2015 at 10:32 AM, Brian Paul <<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.com</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 09/08/2015 08:11 AM, Ilia Mirkin wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
This commit should also remove the relevant test from glean (and<br>
all.py).<br>
</blockquote>
<br>
<br>
Juliet already posted a patch which removes the glean test.  I'm happy to<br>
keep that as a separate change.<br>
</blockquote>
<br>
OK, well, then I assume you'll be comparing the two? I'm definitely<br>
not spending the time searching around for other emails or figuring<br>
out where in glean this came from.<br>
</blockquote>
<br></span>
I'm mentoring Julia for her ESOC project so I'm reviewing her work. Also, I wrote the original glean test.<br>
<br>
Julie, you can mention in the commit message that this new test replaces the original glean tpaths.cpp test.<span class="HOEnZb"><font color="#888888"><br>
<br>
-Brian<br>
<br>
</font></span></blockquote></div><br></div>