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