[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