[chirp_devel] [PATCH] Add test case for settings

Dan Smith
Thu Jan 22 07:50:53 PST 2015


> +        '''
>          _limit = int(self._memobj.limits.lower_vhf) / 10
>          rs = RadioSetting("limits.lower_vhf", "VHF Lower Limit (115-239 MHz)",
>                            RadioSettingValueInteger(115, 235, _limit))
> @@ -429,7 +431,9 @@
>              obj.lower_vhf = value
>          rs.set_apply_callback(apply_limit, self._memobj.limits)
>          basic.append(rs)
> +        '''

It's really confusing to comment out code like this. I'd prefer that we
don't comment out code at all, but if necessary, please use hash marks.
Since we have version control, I'd rather you just remove these if
they're buggy and add them back when they're fixed.

Thanks!

--Dan

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP digital signature
Url : http://intrepid.danplanet.com/pipermail/chirp_devel/attachments/20150122/48554ab0/attachment-0001.bin 


More information about the chirp_devel mailing list