<div dir="ltr">Wrapped up the PEP-8 cleanups as much as possible. There was one where it complained about under indenting, but that's nit picking to the extreme. Caught a few minor issues as well in the process, too.<br></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr">Charles Stewart<br><a href="mailto:ccstewart@gmail.com" target="_blank">chuckination@gmail.com</a></div></div></div>
<br><div class="gmail_quote">On Wed, Apr 1, 2015 at 1:14 AM, Tom Hayward <span dir="ltr"><<a href="mailto:tom@tomh.us" target="_blank">tom@tomh.us</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Mar 31, 2015 at 9:26 PM, Chucks <<a href="mailto:chuckination@gmail.com">chuckination@gmail.com</a>> wrote:<br>
> The __memcache to _memcache change was necessary for the TS2000Radio class<br>
> to be able to access the KenwoodLiveRadio superclass's member variable for<br>
> the subclass's get_memory(), set_memory(), and erase_memory() methods and<br>
> cannot be reverted without impacting functioning of the driver. The double<br>
> underscore is analogous to java/C++ private and the single underscore is<br>
> analogous to protected, and it's part of the python black magic that appears<br>
> stylistic but isn't. I really hate that particular bit of the language<br>
> because it does nothing but cause problems, but there's no such thing as a<br>
> language without warts.<br>
<br>
</span>Okay, that makes sense. I didn't realize you were accessing this from<br>
the subclass.<br>
<br>
P.S. It's not black magic, just name mangling. You should be able to<br>
find it at TS2000Radio._KenwoodLiveRadio__memcache, but please don't<br>
try this :-)<br>
<div class="HOEnZb"><div class="h5"><br>
Tom KD7LXL<br>
_______________________________________________<br>
chirp_devel mailing list<br>
<a href="mailto:chirp_devel@intrepid.danplanet.com">chirp_devel@intrepid.danplanet.com</a><br>
<a href="http://intrepid.danplanet.com/mailman/listinfo/chirp_devel" target="_blank">http://intrepid.danplanet.com/mailman/listinfo/chirp_devel</a><br>
Developer docs: <a href="http://chirp.danplanet.com/projects/chirp/wiki/Developers" target="_blank">http://chirp.danplanet.com/projects/chirp/wiki/Developers</a><br>
</div></div></blockquote></div><br></div>