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.

Charles Stewart
chuckination@gmail.com

On Wed, Apr 1, 2015 at 1:14 AM, Tom Hayward <tom@tomh.us> wrote:
On Tue, Mar 31, 2015 at 9:26 PM, Chucks <chuckination@gmail.com> wrote:
> The __memcache to _memcache change was necessary for the TS2000Radio class
> to be able to access the KenwoodLiveRadio superclass's member variable for
> the subclass's get_memory(), set_memory(), and erase_memory() methods and
> cannot be reverted without impacting functioning of the driver.  The double
> underscore is analogous to java/C++ private and the single underscore is
> analogous to protected, and it's part of the python black magic that appears
> stylistic but isn't.  I really hate that particular bit of the language
> because it does nothing but cause problems, but there's no such thing as a
> language without warts.

Okay, that makes sense. I didn't realize you were accessing this from
the subclass.

P.S. It's not black magic, just name mangling. You should be able to
find it at TS2000Radio._KenwoodLiveRadio__memcache, but please don't
try this :-)

Tom KD7LXL
_______________________________________________
chirp_devel mailing list
chirp_devel@intrepid.danplanet.com
http://intrepid.danplanet.com/mailman/listinfo/chirp_devel
Developer docs: http://chirp.danplanet.com/projects/chirp/wiki/Developers