comparison audio/AudioCallbackPlaySource.h @ 552:8c11ca1ebc39 bqresample

Some fixes to sample rate management

     /**
      * Get the number of channels of audio that will be provided
      * to the play target.  This may be more than the source channel
      * count: for example, a mono source will provide 2 channels
      * after pan.
+     *
      * This may safely be called from a realtime thread.  Returns 0 if
      * there is no source yet available.
-     */
-    int getTargetChannelCount() const;
+     *
+     * override from AudioPlaySource
+     */
+    virtual int getTargetChannelCount() const override;
 
     /**
      * ApplicationPlaybackSource equivalent of the above.
-     */
-    virtual int getApplicationChannelCount() const {
+     *
+     * override from breakfastquay::ApplicationPlaybackSource
+     */
+    virtual int getApplicationChannelCount() const override {
         return getTargetChannelCount();
     }
     
     /**
-     * Get the actual sample rate of the source material.  This may
-     * safely be called from a realtime thread.  Returns 0 if there is
-     * no source yet available.
-     */
-    virtual sv_samplerate_t getSourceSampleRate() const;
-
-    /**
-     * ApplicationPlaybackSource equivalent of the above.
-     */
-    virtual int getApplicationSampleRate() const {
-        return int(round(getSourceSampleRate()));
+     * Get the actual sample rate of the source material (the main
+     * model).  This may safely be called from a realtime thread.
+     * Returns 0 if there is no source yet available.
+     *
+     * When this changes, the AudioCallbackPlaySource notifies its
+     * ResamplerWrapper of the new sample rate so that it can resample
+     * correctly on the way to the device (which is opened at a fixed
+     * rate, see getApplicationSampleRate).
+     */
+    virtual sv_samplerate_t getSourceSampleRate() const override;
+
+    /**
+     * ApplicationPlaybackSource interface method: get the sample rate
+     * at which the application wants the device to be opened. We
+     * always allow the device to open at its default rate, and then
+     * we resample if the audio is at a different rate. This avoids
+     * having to close and re-open the device to obtain consistent
+     * behaviour for consecutive sessions with different source rates.
+     */
+    virtual int getApplicationSampleRate() const override {
+        return 0;
     }
 
     /**
      * Get "count" samples (at the target sample rate) of the mixed
      * audio data, in all channels.  This may safely be called from a