Update the resampler logic to follow that of R2 more - ignore the pitch hq/hs setting in offline mode entirely. The alternative is too tricky to handle elegantly in the command line tool, which I think means too complicated

rubberband/RubberBandStretcher.h

@@ -328,10 +328,10 @@ public:
      *   perceived pitch profile of the voice or instrument.
      *
      * 10. Flags prefixed \c OptionPitch control the method used for
-     * pitch shifting. In the R2 engine they may be changed at any
+     * pitch shifting. These options affect only realtime mode. In
-     * time but affect only realtime mode (in offline mode the method
+     * offline mode the method is not adjustable. In the R2 engine
-     * cannot be changed). In the R3 engine they affect both realtime
+     * these options may be changed at any time; in the R3 engine they
-     * and offline modes but are fixed on construction.
+     * may be set only on construction.
      *
      *   \li \c OptionPitchHighSpeed - Favour CPU cost over sound
      *   quality. This is the default. Use this when time-stretching

src/finer/R3Stretcher.cpp

@@ -274,13 +274,7 @@ R3Stretcher::createResampler()
     Profiler profiler("R3Stretcher::createResampler");
     
     Resampler::Parameters resamplerParameters;
-
+    resamplerParameters.quality = Resampler::FastestTolerable;
-    if (m_parameters.options & RubberBandStretcher::OptionPitchHighQuality) {
-        resamplerParameters.quality = Resampler::Best;
-    } else {
-        resamplerParameters.quality = Resampler::FastestTolerable;
-    }
-    
     resamplerParameters.initialSampleRate = m_parameters.sampleRate;
     resamplerParameters.maxBufferSize = m_guideConfiguration.longestFftSize;
 

src/finer/R3Stretcher.h

@@ -418,21 +418,53 @@ protected:
 
         if (before) *before = false;
         if (after) *after = false;
+
+        // Some of the logic about when/whether to resample may have
+        // already been decided, because we might not have a
+        // resampler. This call is for the processing path, we don't
+        // make big decisions here and can't be saying we should
+        // resample if there is no resampler
         if (!m_resampler) return;
 
-        if (m_parameters.options &
+        // In offline mode we always ignore the OptionPitch setting
-            RubberBandStretcher::OptionPitchHighConsistency) {
+        // and resample afterwards, like OptionPitchHighConsistency
+        // (except that we don't resample for ratio of 1.0). This is
+        // because (a) the initial algorithm was developed and tested
+        // that way, (b) R2 works that way and nobody has ever
+        // complained, and (c) otherwise for the command-line tool
+        // (which is probably the most common way to use RB offline)
+        // we would have to choose between defaulting to lower-quality
+        // OptionPitchHighSpeed or offering yet another setting, both
+        // of which are undesirable.
+        
+        if (!isRealTime()) {
+            if (m_pitchScale != 1.0) {
+                // Offline, any pitch scale
+                if (after) *after = true;
+            }
+        } else if (m_parameters.options &
+                   RubberBandStretcher::OptionPitchHighConsistency) {
+            // RT, any pitch scale, HC
             if (after) *after = true;
             
         } else if (m_pitchScale != 1.0) {
-            if (m_pitchScale > 1.0 &&
+
-                (m_parameters.options &
+            bool hq = (m_parameters.options &
-                 RubberBandStretcher::OptionPitchHighQuality)) {
+                       RubberBandStretcher::OptionPitchHighQuality);
-                if (after) *after = true;
+            // We already handled OptionPitchHighConsistency, so if
+            // !hq then we must be HighSpeed
+            if (m_pitchScale > 1.0) {
+                if (hq) {
+                    if (after) *after = true;
+                } else {
+                    if (before) *before = true;
+                }
             } else if (m_pitchScale < 1.0) {
-                if (after) *after = true;
+                if (hq) {
-            } else {
+                    if (before) *before = true;
-                if (before) *before = true;
+                } else {
+                    if (after) *after = true;
+                }
             }
         }
     }