Re: [chirp_devel] [PATCH] [uv-b5] Add 'off' to the list of Duplex options

@@ -290,6 +294,10 @@ _mem.set_raw("\xFF" * 16) return

•    if mem.duplex == "off":
  

•        mem.offset = mem.freq
  

•        mem.duplex = "-"
  

This modifies the object that was being passed in, which is "bad". It won't technically hurt anything when using the GUI, but it could have other ramifications if the driver is used for something else.

Instead, can you just handle this near where the _mem object is changed, like this:

if mem.duplex == "off": _mem.duplex = DUPLEX.index("-") _mem.offset = _mem.freq else: _mem.offset = mem.offset / 10 _mem.duplex = DUPLEX.index(mem.duplex)