[bitbake-devel] [PATCH] fetch/git: Separate out an ls-remote function

Chris Larson clarson at kergoth.com
Sun Mar 2 04:50:16 UTC 2014


On Fri, Feb 28, 2014 at 10:22 AM, Richard Purdie <
richard.purdie at linuxfoundation.org> wrote:

> -    def _latest_revision(self, ud, d, name):
> +    def __lsremote(self, ud, d, search):
>

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 :)
-- 
Christopher Larson
clarson at kergoth dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/bitbake-devel/attachments/20140301/f7486cce/attachment-0002.html>


More information about the bitbake-devel mailing list