Improve WavPack decoder performance

[sbooth]

No description provided.

[github-actions]

Cpp-linter Review

Used clang-format v22.1.6

Click here for the full clang-format patch

diff --git a/Sources/CSFBAudioEngine/Decoders/SFBWavPackDecoder.m b/Sources/CSFBAudioEngine/Decoders/SFBWavPackDecoder.m
index b6c0217..74a5d9a 100644
--- a/Sources/CSFBAudioEngine/Decoders/SFBWavPackDecoder.m
+++ b/Sources/CSFBAudioEngine/Decoders/SFBWavPackDecoder.m
@@ -331 +331 @@ static int can_seek_callback(void *id) {
-    //int qmode = WavpackGetQualifyMode(_wpc);
+    // int qmode = WavpackGetQualifyMode(_wpc);
@@ -337 +337 @@ static int can_seek_callback(void *id) {
-    //} else if(qmode & QMODE_DSD_AUDIO) {
+        //} else if(qmode & QMODE_DSD_AUDIO) {
@@ -467 +467 @@ static int can_seek_callback(void *id) {
-        //int qmode = WavpackGetQualifyMode(_wpc);
+        // int qmode = WavpackGetQualifyMode(_wpc);
@@ -470 +470,2 @@ static int can_seek_callback(void *id) {
-            memcpy((unsigned char *)buffer.audioBufferList->mBuffers[0].mData + buffer.audioBufferList->mBuffers[0].mDataByteSize,
+            memcpy((unsigned char *)buffer.audioBufferList->mBuffers[0].mData +
+                           buffer.audioBufferList->mBuffers[0].mDataByteSize,
@@ -480 +481,2 @@ static int can_seek_callback(void *id) {
-                uint32_t *dst = (uint32_t *)((unsigned char *)buffer.audioBufferList->mBuffers[0].mData + buffer.audioBufferList->mBuffers[0].mDataByteSize);
+                uint32_t *dst = (uint32_t *)((unsigned char *)buffer.audioBufferList->mBuffers[0].mData +
+                                             buffer.audioBufferList->mBuffers[0].mDataByteSize);
@@ -496 +498,2 @@ static int can_seek_callback(void *id) {
-                memcpy((unsigned char *)buffer.audioBufferList->mBuffers[0].mData + buffer.audioBufferList->mBuffers[0].mDataByteSize,
+                memcpy((unsigned char *)buffer.audioBufferList->mBuffers[0].mData +
+                               buffer.audioBufferList->mBuffers[0].mDataByteSize,
@@ -501 +504,2 @@ static int can_seek_callback(void *id) {
-            float *dst = (float *)((unsigned char *)buffer.audioBufferList->mBuffers[0].mData + buffer.audioBufferList->mBuffers[0].mDataByteSize);
+            float *dst = (float *)((unsigned char *)buffer.audioBufferList->mBuffers[0].mData +
+                                   buffer.audioBufferList->mBuffers[0].mDataByteSize);
@@ -507 +511 @@ static int can_seek_callback(void *id) {
-            //float scaleFactor = ldexpf(1.0f, -(WavpackGetBitsPerSample(_wpc) - 1));
+            // float scaleFactor = ldexpf(1.0f, -(WavpackGetBitsPerSample(_wpc) - 1));

Have any feedback or feature suggestions? Share it here.

outdated suggestion

[Copilot]

Pull request overview

This PR aims to improve WavPack decoding throughput by switching the decoder’s PCM output to interleaved buffers and using bulk/vectorized operations (memcpy, SIMD, vDSP) instead of per-channel deinterleaving loops.

Changes:

• Switch WavPack decoder output formats from non-interleaved to interleaved for float and integer PCM.
• Replace manual deinterleaving/loops with bulk memcpy, SIMD shifting for lossless integer alignment, and Accelerate/vDSP for lossy int→float conversion.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.