<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 21, 2014 at 9:46 AM, Jacob Kroon <span dir="ltr">&lt;<a href="mailto:jacob.kroon@mikrodidakt.se" target="_blank">jacob.kroon@mikrodidakt.se</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":12f" class="a3s" style="overflow:hidden">-            funcname = (&quot;__anon_%s_%s&quot; % (self.lineno, self.filename.translate(string.maketrans(&#39;/.+-@&#39;, &#39;_____&#39;))))<br>


+            funcname = (&quot;__anon_%s_%s&quot; % (self.lineno, self.filename.translate(string.maketrans(&#39;/.+-@%&#39;, &#39;______&#39;))))</div></blockquote></div><br></div><div class="gmail_extra">I know it&#39;s not related to this particular patch, but there&#39;s no reason for us to call string.maketrans() inside of MethodPool.eval(), that should probably have been a class member, no reason to regenerate the translation table every time, after all :)<br>

-- <br>Christopher Larson<br>clarson at kergoth dot com<br>Founder - BitBake, OpenEmbedded, OpenZaurus<br>Maintainer - Tslib<br>Senior Software Engineer, Mentor Graphics
</div></div>