[chirp_devel] [PATCH 1 of 1] [ft8x7] pad labels with blanks

Marco Filippi IZ3GME
Wed Jun 26 07:52:58 PDT 2013


# HG changeset patch
# User Marco Filippi <iz3gme.marco at gmail.com>
# Date 1372258225 -7200
# Node ID 7f70ba4a94d38bb52627915c50c006fc0336ea33
# Parent  598c397d39c0ab9366256a4fef6ef9e7d426ec50
[ft8x7] pad labels with blanks
the bug seems to be limited to a few production lot of ft857d but padding
is safe for all radios in ft8x7 family and it can avoid future problems
Fix #937

diff --git a/chirp/ft817.py b/chirp/ft817.py
--- a/chirp/ft817.py
+++ b/chirp/ft817.py
@@ -687,12 +687,11 @@
         _mem.rit = 0	# not supported in chirp
         _mem.freq = mem.freq / 10
         _mem.offset = mem.offset / 10
-        for i in range(0, min(len(mem.name.rstrip()),8)):
-            _mem.name[i] = ord(mem.name[i])
-        # rfill with 0xff to mimic radio
-        # "quick&dirty" use of i from previous cicle 
-        for i in range(i+1,8):
-            _mem.name[i] = 0xff
+        # there are ft857D that have problems with short labels, see bug #937
+        # some of the radio fill with 0xff and some with blanks
+        # the latter is safe for all ft8x7 radio so why should i do it only for some?
+        for i in range(0, 8):
+            _mem.name[i] = self.CHARSET.index(mem.name.ljust(8)[i])
         
         for setting in mem.extra:
             setattr(_mem, setting.get_name(), setting.value)



More information about the chirp_devel mailing list