Re: [chirp_devel] [PATCH] Fix logic error preventing auto repeater setting in some cases

If you drop self._config.get_bool("autorpt"), this reads:

I'm not sure why dropping it has any relevance, but perhaps you're just saying it reads oddly?

•    is_changed = new != prev if was_filled else True
  

•    if new and is_changed:
  

How about we change it to:

is_changed = new != prev if was_filled else True autorpt_enabled = self._config.get_bool("autorpt")

if new is not None and is_changed and autorpt_enabled: ....

?

In reality, that's why new is in the comparison (since it could be set to None right above in the parse_freq() bit). Perhaps the more explicit "is not None" reads better?