[chirp_devel] Issue with Baofeng Firmware Version
Jim Unroe
Sat Feb 13 18:24:33 PST 2016
> So it sounds like we should be skipping any immutable fields during
> set_settings(). So around here:
>
>> if element.has_apply_callback():
>> LOG.debug("Using apply callback")
>> element.run_apply_callback()
>> else:
>> LOG.debug("Setting %s = %s" % (setting, element.value))
>> setattr(obj, setting, element.value)
>
>
> What if we changed that else to "elif setting._mutable:" ?
>
> Then it would never run the "set" on immutable fields, which is probably
> what we should have done in the first place.
>
> Does that work?
>
Makes sense. I will have the person with the issue test it.
Thanks,
Jim
More information about the chirp_devel
mailing list