[chirp_devel] [PATCH] [uv5r]Add Shift and Offset support to Work Mode VFO Presets

[Jim Unroe]

>
>
> I've already applied this, but could you follow up with a patch to
> remove all the debug print statements that we've added while working on
> this? The UV5R driver is really chatty and I think we can clean up most
> of that stuff now that we know it's working. Just if/when you have
> time, I'd appreciate it.
>

Dang it. I meant to do that. Sorry. It shall be done.


>
> I'm also a little concerned at the size of the get_settings() function
> now that you've gone and added every dang feature the radio supports :)
>
> I think we should make a point of going back and splitting these out
> into separate functions, just for cleanliness. It won't really affect
> how it works, just how it looks to the developer. I think maybe one
> helper function per page would be fine.


> Are you interested in helping with this? I don't want to derail your
> progress towards getting anything else closed out, but definitely don't
> want it to get away from us :)
>

Sure. But I'm not sure exactly what you are looking for. But I'm pretty
sure that once you get me started, I can help with this.


>
> Thanks!
>
>
>
You're welcome

Jim
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://intrepid.danplanet.com/pipermail/chirp_devel/attachments/20130219/5aebfb81/attachment-0001.html