Bug #7601

[Yaesu FT-4, FT-65, FT-25] Sometimes channel names are not displayed correctly

Added by Bernhard Hailer almost 3 years ago. Updated almost 3 years ago.

Status:Closed Start date:01/24/2020
Priority:Normal Due date:
Assignee:Bernhard Hailer % Done:

100%
Category:-
Target version:chirp-daily
Chirp Version:daily Platform:All
Model affected:Yaesu FT-4, FT-65, FT-25

Description

If user programs radio manually and adds a name shorter than 8 characters, the remaining characters are filled with garbage on the Chirp UI.

Associated revisions

Revision 3298:d841aaba1977
Added by Bernhard Hailer almost 3 years ago

[ft4] driver restructure (#7615)

While implementing two more radios of the Yaesu FT-4 family (the FT-25, see #7543, and the FT-4V, see #7387), I found that with some moderate reorganization, implementation of new radios will become easier. The proposed reorganization implements one more interstitial layer of inheritance to support the sub families of FT-4 (containing the FT-4X and FT-4V) and FT-65 (containing FT-65 and FT-25). Also, some variable assignments have been moved from the individual radio classes to the SCU-35 base class and to the interstitial classes named above.

This change also adds the infrastructure for adding European or Asian models, and as such prepare for addressing a number of currently open issues (6.25kHz tune step issues on EU models, frequency limitations; see #6619, #6651, #6677, #6761, #6869). It will also make it easier for a few additional fixes for issues which were found during my work on this driver (#7601, #7603, #7605). I will submit these fixes in additional patches.

Fixes: #7615

Revision 3304:918312d3e8c8
Added by Bernhard Hailer almost 3 years ago

[ft4] Sometimes channel names are not displayed correctly. Fixes: #7601
When user programmed radio manually and added a name shorter than 8 characters, the remaining characters were filled with garbage on the Chirp UI. This was due to padding with 0x7F used by the Yaesu; the solution is to pad with 0x20 (space characters) instead.

History

#1
Updated by Bernhard Hailer almost 3 years ago

  • Status changed from New to In Progress

I will prepare a set of patches this weekend.
A build should be available soon.

#2
Updated by Anonymous almost 3 years ago

  • Status changed from In Progress to Closed

Applied in changeset 918312d3e8c8.

Also available in: Atom PDF