Add freq to lanentry
Fix handling of missing QRG for log entry sent via LAN.
Frequency is correctly stored, but see the comments.
Thanks for review. See answers inline.
How about this plan?
- in this PR keep only the minimal changes to get freq into the log and also remove the code for dropping the last char of the LAN message.
- document LAN messages in the manual
- test to see if messages work as required and fix the arising issues.
I can help with 2) and 3).
See the last commit to see a more developed state after harmonizing the sent code.
But, let us discus your plan.
1. in this PR keep only the minimal changes to get freq into the log and also remove the code for dropping the last char of the LAN message.
Ok, rework of the commits can be made easily. I would make a new PR for it. (Maybe not before friday). But I would still recommend to send ALL message NL terminated in the end (makes debugging more easy). So no need to drop the last char.
2. document LAN messages in the manual
Or maybe better in the code, e.g. in lancode.h. I think the message format is not relevant for normal user.
3. test to see if messages work as required and fix the arising issues.I can help with 2) and 3).
Fine by me.
Thanks for the updates. I still see no need for the NL, but can live with it. You can use this PR, no separate one is needed IMO.
Re 2): lancode.h is also a perfect place to document the messages.
That should be step 1. Using g_strchomp is a good idea.
Thanks for the update.