31 Dec
2014
31 Dec
'14
11:50 p.m.
2014-12-31 15:25 GMT-08:00 jml jml@jmlzone.com:
I know this is a small detail, but you need # in front of the issue number for it to be detected correctly.
You have some lines greater than 79 characters. These should be wrapped. https://www.python.org/dev/peps/pep-0008/#maximum-line-length
The patch is really funky and hard to read--it intermingles _upload() and old set_memory(). I'll have to import this before I can comment on much more.
Glad to see someone picking up support for this radio. There seems to be a lot of demand.
Tom KD7LXL