pflarue / ardop

Source for various version of ARDOP
Other
23 stars 4 forks source link

Fix rare bug the caused PTT stuck on #25

Closed pflarue closed 1 month ago

pflarue commented 1 month ago

Recent changes created a bug that left PTT stuck on, and made ardopcf unable to be terminated with Ctrl-C. Luckily, it wasn't something that would be triggered easily by most users. However, fixing it demonstrated the lack of fault checking in the the code for modulation and transmit. So, as a part of this fix, I added more fault checking, so that future failures should be less dramatic. The error messages from these checks are also intended to help users who encounter any of them to provide a useful report describing the problem, even if they don't know how to reproduce it.