<div class="gmail_quote">On Tue, May 25, 2010 at 1:27 PM, detlev <span dir="ltr">&lt;<a href="mailto:detlev@die-offenbachs.de">detlev@die-offenbachs.de</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div><div></div><div class="h5"><br>
</div></div>Thanks for the patch. The code was back-ported from eric5 and I missed this<br>
place (it&#39;s the fact that QString is gone in PyQt4 API v2)<br>
<div><div></div><div class="h5"><br></div></div></blockquote></div>That is most probably a nit picking and a matter of taste but the name &#39;dir&#39; for a<br>local variable is a little bit confusing. In fact because I&#39;m completely unfamiliar <br>
with the PyQt4 API I&#39;ve bluntly  inserted &#39;print dir(self.dirHistory)&#39; after <br>&#39;if self.dirButton.isChecked():&#39; to see what attributes are available and immediately<br>got an error &quot;using local variable &#39;dir&#39; before assignment&quot;. The better name would <br>
be something like &quot;currentDir&quot;. IMHO of course.<br><br>Thanks for the quick fix!<br><br>Regards,<br>-- <br>Mikhail Terekhov<br>