rigctld: Fix sending cw and packet handlings

Sunguk Leerequested to merge
d3m3vilurr/wfview:fix-rigctl-command-handlings into master

this patchset contains fixing various wrong behaviors

  • sending cw was only supported with short command mode. this issue was introduced since 9939a415 ; resolve #173

  • short command always need to remove first char. imho, it's another 9939a415's bug. I think rigctl protocol is designed command + empty_space additional params\n (like b message\n) but after the patch, now support without space.

    IDK it's correct or not, so a patch will support both of them now.

  • cannot detect some short command which has +127 value. sstr type is uchar. but imo, commands[0].toLatin1() would return char or unicode.

    I didn't do digging it, I just change it to casting form.

  • remove custom handling to exit or rerun the commands loop

    these handling logic were ignored some next commands, missing returns, and possibly makes buggy behavior.

please, feel free to review and merge.

73 DE DS1UFX

Edited by Sunguk Lee

Merge request reports