fix(processor): use univector as buffers

Author: zsliu98

source/PluginProcessor.cpp

@@ -96,14 +96,20 @@ void PluginProcessor::prepareToPlay(const double sample_rate, const int samples_
         static_cast<juce::uint32>(samples_per_block),
         2
     };
-    float_buffer_.setSize(4, samples_per_block);
-    float_buffer_.clear();
-    float_side_pointers_[0] = float_buffer_.getWritePointer(2);
-    float_side_pointers_[1] = float_buffer_.getWritePointer(3);
-    double_buffer_.setSize(2, samples_per_block);
-    double_side_pointers_[0] = double_buffer_.getWritePointer(0);
-    double_side_pointers_[1] = double_buffer_.getWritePointer(1);
-    double_buffer_.clear();
+    const auto buffer_size = static_cast<size_t>(samples_per_block);
+
+    float_buffer_[0].resize(buffer_size);
+    float_buffer_[1].resize(buffer_size);
+    float_buffer_[2].resize(buffer_size);
+    float_buffer_[3].resize(buffer_size);
+    float_side_pointers_[0] = float_buffer_[2].data();
+    float_side_pointers_[1] = float_buffer_[3].data();
+
+    double_buffer_[0].resize(buffer_size);
+    double_buffer_[1].resize(buffer_size);
+    double_side_pointers_[0] = double_buffer_[0].data();
+    double_side_pointers_[1] = double_buffer_[1].data();
+
     compressor_controller_.prepare(spec);
     // determine current channel layout
     const auto *main_bus = getBus(true, 0);
@@ -154,13 +160,14 @@ bool PluginProcessor::isBusesLayoutSupported(const BusesLayout &layouts) const {
 
 void PluginProcessor::processBlock(juce::AudioBuffer<float> &buffer, juce::MidiBuffer &) {
     juce::ScopedNoDenormals no_denormals;
-    if (buffer.getNumSamples() == 0) return; // ignore empty blocks
+    if (buffer.getNumSamples() == 0) { return; } // ignore empty blocks
     const auto c_use_ext_side = use_ext_side_.load(std::memory_order::relaxed);
     const auto buffer_size = static_cast<size_t>(buffer.getNumSamples());
     switch (channel_layout_) {
         case ChannelLayout::kMain1Aux0: {
+            if (buffer.getNumChannels() < 1) { return; }
             main_pointers_[0] = buffer.getWritePointer(0);
-            main_pointers_[1] = float_buffer_.getWritePointer(1);
+            main_pointers_[1] = float_buffer_[1].data();
             zldsp::vector::copy(main_pointers_[1], main_pointers_[0], buffer_size);
 
             zldsp::vector::copy<double, float>(double_side_pointers_, main_pointers_, buffer_size);
@@ -171,8 +178,9 @@ void PluginProcessor::processBlock(juce::AudioBuffer<float> &buffer, juce::MidiB
             break;
         }
         case ChannelLayout::kMain1Aux1: {
+            if (buffer.getNumChannels() < 2) { return; }
             main_pointers_[0] = buffer.getWritePointer(0);
-            main_pointers_[1] = float_buffer_.getWritePointer(1);
+            main_pointers_[1] = float_buffer_[1].data();
             zldsp::vector::copy(main_pointers_[1], main_pointers_[0], buffer_size);
 
             if (c_use_ext_side) {
@@ -188,8 +196,9 @@ void PluginProcessor::processBlock(juce::AudioBuffer<float> &buffer, juce::MidiB
             break;
         }
         case ChannelLayout::kMain1Aux2: {
+            if (buffer.getNumChannels() < 3) { return; }
             main_pointers_[0] = buffer.getWritePointer(0);
-            main_pointers_[1] = float_buffer_.getWritePointer(1);
+            main_pointers_[1] = float_buffer_[1].data();
             zldsp::vector::copy(main_pointers_[1], main_pointers_[0], buffer_size);
 
             if (c_use_ext_side) {
@@ -205,6 +214,7 @@ void PluginProcessor::processBlock(juce::AudioBuffer<float> &buffer, juce::MidiB
             break;
         }
         case ChannelLayout::kMain2Aux0: {
+            if (buffer.getNumChannels() < 2) { return; }
             main_pointers_[0] = buffer.getWritePointer(0);
             main_pointers_[1] = buffer.getWritePointer(1);
 
@@ -216,6 +226,7 @@ void PluginProcessor::processBlock(juce::AudioBuffer<float> &buffer, juce::MidiB
             break;
         }
         case ChannelLayout::kMain2Aux1: {
+            if (buffer.getNumChannels() < 3) { return; }
             main_pointers_[0] = buffer.getWritePointer(0);
             main_pointers_[1] = buffer.getWritePointer(1);
 
@@ -232,6 +243,7 @@ void PluginProcessor::processBlock(juce::AudioBuffer<float> &buffer, juce::MidiB
             break;
         }
         case ChannelLayout::kMain2Aux2: {
+            if (buffer.getNumChannels() < 4) { return; }
             main_pointers_[0] = buffer.getWritePointer(0);
             main_pointers_[1] = buffer.getWritePointer(1);
 

source/PluginProcessor.hpp

@@ -81,8 +81,8 @@ class PluginProcessor : public juce::AudioProcessor {
 private:
     zlp::CompressorController compressor_controller_;
     zlp::CompressAttach compress_attach_;
-    juce::AudioBuffer<float> float_buffer_;
-    juce::AudioBuffer<double> double_buffer_;
+    std::array<kfr::univector<float>, 4> float_buffer_;
+    std::array<kfr::univector<double>, 2> double_buffer_;
     std::array<float *, 2> main_pointers_{}, float_side_pointers_{};
     std::array<double *, 2> double_side_pointers_{};