Hi,
Attached is patch "vx8-4881-4.patch" which corrects the line > 79 chars violation. Also, I discovered that I left out the pad value of one of the new #seekto format strings I had added (had 0x%0X instead of 0x%04X). That's also corrected with this patch.
Cheers,
Keith
On Tue, Jun 6, 2017 at 7:32 PM, Keith Williamson hkwilliamson@gmail.com wrote:
Ah, I see. I figured if a patch "failed" the build it was yanked.
No problem. I'll send a patch that just fixes the PEP-8 error.
On Tue, Jun 6, 2017 at 6:14 PM, Dan Smith via chirp_devel < chirp_devel@intrepid.danplanet.com> wrote:
Resubmitting "vx8-4881-2.patch" as "vx8-4881-3.patch" because of PEP-8 error due to line length of 80. My bad..thought max line length was 80. Ensured that all lines are strictly less than 80 chars.
It's already in tree, so I can't apply this unless I revert the other one. Can you just send a teensy incremental change rebased on the tip?
Thanks!
--Dan _______________________________________________ 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