<div dir="ltr">Just an oversight, will fix.<div><br></div><div>Thanks!</div><div> - Daniel</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Aug 7, 2013 at 2:50 PM, Eli Bendersky <span dir="ltr"><<a href="mailto:eliben@google.com" target="_blank">eliben@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote"><div class="im">On Wed, Aug 7, 2013 at 2:43 PM, Daniel Dunbar <span dir="ltr"><<a href="mailto:daniel@zuster.org" target="_blank">daniel@zuster.org</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: ddunbar<br>
Date: Wed Aug  7 16:43:17 2013<br>
New Revision: 187918<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=187918&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=187918&view=rev</a><br>
Log:<br>
[lit] Use list comprehensions instead of map().<br>
<br>
Modified:<br>
    llvm/trunk/utils/lit/lit/LitConfig.py<br>
    llvm/trunk/utils/lit/lit/TestRunner.py<br>
    llvm/trunk/utils/lit/lit/__init__.py<br>
<br>
Modified: llvm/trunk/utils/lit/lit/LitConfig.py<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/lit/lit/LitConfig.py?rev=187918&r1=187917&r2=187918&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/lit/lit/LitConfig.py?rev=187918&r1=187917&r2=187918&view=diff</a><br>



==============================================================================<br>
--- llvm/trunk/utils/lit/lit/LitConfig.py (original)<br>
+++ llvm/trunk/utils/lit/lit/LitConfig.py Wed Aug  7 16:43:17 2013<br>
@@ -32,7 +32,7 @@ class LitConfig:<br>
         # The name of the test runner.<br>
         self.progname = progname<br>
         # The items to add to the PATH environment variable.<br>
-        self.path = list(map(str, path))<br>
+        self.path = list([str(p) for p in path])<br></blockquote><div><br></div></div><div>Here (and maybe elsewhere) - what is the purpose of the extra list() call?</div><span class="HOEnZb"><font color="#888888"><div>
<br></div><div>Eli</div><div><br></div></font></span></div></div>

</div>
</blockquote></div><br></div>