Use unique_ptr for m_outBuf

sakertooth · Aug 11, 2024 · 261642d · 261642d
1 parent 12b9de4
commit 261642d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/include/AudioPortAudio.h b/include/AudioPortAudio.h
@@ -90,7 +90,7 @@ class AudioPortAudio : public AudioDevice
 
 	bool m_wasPAInitError;
 
-	SampleFrame* m_outBuf;
+	std::unique_ptr<SampleFrame[]> m_outBuf;
 	std::size_t m_outBufPos;
 	fpp_t m_outBufSize;
 

diff --git a/src/core/audio/AudioPortAudio.cpp b/src/core/audio/AudioPortAudio.cpp
@@ -45,7 +45,7 @@ AudioPortAudio::AudioPortAudio(bool& successful, AudioEngine* engine)
 		  engine)
 	, m_paStream(nullptr)
 	, m_wasPAInitError(false)
-	, m_outBuf(new SampleFrame[engine->framesPerPeriod()])
+	, m_outBuf(std::make_unique<SampleFrame[]>(engine->framesPerPeriod()))
 	, m_outBufPos(0)
 	, m_outBufSize(engine->framesPerPeriod())
 {
@@ -139,7 +139,6 @@ AudioPortAudio::~AudioPortAudio()
 	{
 		Pa_Terminate();
 	}
-	delete[] m_outBuf;
 }
 
 
@@ -186,7 +185,7 @@ int AudioPortAudio::processCallback(const float* inputBuffer, float* outputBuffe
 	{
 		if( m_outBufPos == 0 )
 		{
-			const fpp_t frames = getNextBuffer( m_outBuf );
+			const fpp_t frames = getNextBuffer(m_outBuf.get());
 			if( !frames )
 			{
 				m_stopped = true;