Radioddity FS-T3 family lacks roger beep capability. To avoid confusing users, in drivers/radtel_t18.py I enclosed the roger beep setting in an "if" statement. That meant it needed another level of indenting.
if self.MODEL != "BF-T12" and self.MODEL != "FS-T3" and \ self.MODEL != "PR-T3": rs = RadioSetting("rogerbeep", "Roger beep", RadioSettingValueBoolean(_settings.rogerbeep)) basic.append(rs)
(Pretend there are 12 more spaces at the start of each line.)
Ran "tox -e style" to test for code standards. Error message:
./chirp/drivers/radtel_t18.py:873:80: E501 line too long (80 > 79 characters)
One character too long! Backed off the RadioSettingValueBoolean indent by one space. Ran it again:
./chirp/drivers/radtel_t18.py:873:34: E128 continuation line under-indented for visual indent
So it can meet style standards for line length or visual indent, but not both at the same time. Tried to trick it by adding \ to the previous line, turning it into a continuation that might have different indent rules. Didn't work.
./chirp/drivers/radtel_t18.py:872:62: E502 the backslash is redundant between brackets
How much of a problem is this really? Any suggestions to make it pass tests?
---- Charles Terrell tachyon@pobox.com