This is necessary to keep alignment between the client view of rig->state and the shared libarary view
Including PTHREAD functions in rig->state necessitated this as HAVE_PTHREAD only defined during hamlib compilation.
Clients including hamlib/rig.h did not pick up all the HAVE* config variable that control what gets included
https://github.com/Hamlib/Hamlib/issues/947
We only look now for 0x0a on commands that expect a return
If there is a rig/emulator that does not give 0x0a on expected returns then we will have a problem. gs232a/gs232b doc says 0x0a is expected on
cmds that return info and 0x0d on command with no return.
https://github.com/Hamlib/Hamlib/issues/371
Fixed using the following command:
codespell --write-changes --summary --skip=*.m4 --ignore-words-list="develope,get's,quitt,setts,som,ue,vektor"
codespell --write-changes --summary --skip=aclocal.m4,lib --ignore-words-list="develope,get's,quitt,setts,som,ue,vektor"
Codespell home page: https://github.com/codespell-project/codespell
This covers the 12PR1A rotor which claims gs232b compatibility
The 12PR1A also claims gs232a compability which is wrong as the AZ= EL= format is gs232b
https://github.com/Hamlib/Hamlib/issues/309
Simplify gs232a parsing but mandate both arguments in +0xxx+0xxx format