No error checking / validation of sample rates about soapyairspy HOT 2 OPEN

And I think because of how the library is, the call cannot be made during setSampleRate() because that would be an issue while streaming.

One solution might be to simply check the rate passed into setSampleRate() against the values returned by airspy_get_samplerates. I believe the airspy code basically replicates airspy_get_samplerates inside of airspy_set_samplerate anyway.

Another solution might be to move this whole chunk of code into the setSampleRate() and handle the error code there

if (airspy_is_streaming(dev))
{
        airspy_stop_rx(dev);
        r = airspy_set_samplerate(dev, sampleRate);
        airspy_start_rx(dev, &_rx_callback, (void *) this);
 }
else  r = airspy_set_samplerate(dev, sampleRate);
//throw if r is bad

If you have the hardware available, do you want to try out a fix and make a PR? I dont think this is a change that can be done blindly (ie it just compiles).

from soapyairspy.

I do have hardware, I'll have an experiment.

(From looking through the library/firmware, I think that maybe it's safe to call airspy_set_samplerate while streaming data; the firmware seems to have the right logic to stop/retune/restart; I'll try that too)

from soapyairspy.