I added OPON to change operators (same command as N1MM for ops that use both) and also added CWON and CWOFF commands. I also changed the code to a case statement for readability.
I also made it so when ALT-K is used the CW speed changes to NO CW so the user has an indication that no CW will be sent. This is same display in Toggle CW. Toggle CW checked just CWEnabled instead of CWEnable.
We seem to have two Booleans that indicate if CW is auto sent. This should be changed to only use a single variable. Otherwise it is easy to confuse.
n4af
commented
3 years ago
I added OPON to change operators (same command as N1MM for ops that use both) and also added CWON and CWOFF commands. I also changed the code to a case statement for readability.
I also made it so when ALT-K is used the CW speed changes to NO CW so the user has an indication that no CW will be sent. This is same display in Toggle CW. Toggle CW checked just CWEnabled instead of CWEnable. We seem to have two Booleans that indicate if CW is auto sent. This should be changed to only use a single variable. Otherwise it is easy to confuse.