<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 28, 2014 at 10:22 AM, Richard Purdie <span dir="ltr"><<a href="mailto:richard.purdie@linuxfoundation.org" target="_blank">richard.purdie@linuxfoundation.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":1de" class="a3s" style="overflow:hidden">- def _latest_revision(self, ud, d, name):<br>
+ def __lsremote(self, ud, d, search):</div></blockquote></div><br></div><div class="gmail_extra">Are we sure we want to use __ here? Double-underscore will trigger python internal name mangling. While using it doesn't break anything, it doesn't really buy us much either vs _, and makes inspection of the class/object rather ugly, IMO :)<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>