Only set kAudioFormatFlagIsAlignedHigh when applicable

[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/SFBFLACDecoder.mm b/Sources/CSFBAudioEngine/Decoders/SFBFLACDecoder.mm
index 6f027ea..62a4d40 100644
--- a/Sources/CSFBAudioEngine/Decoders/SFBFLACDecoder.mm
+++ b/Sources/CSFBAudioEngine/Decoders/SFBFLACDecoder.mm
@@ -298,2 +298,2 @@ void errorCallback(const FLAC__StreamDecoder *decoder, FLAC__StreamDecoderErrorS
-    processingStreamDescription.mFormatFlags = kAudioFormatFlagsNativeEndian | kAudioFormatFlagIsSignedInteger |
-                                               kAudioFormatFlagIsNonInterleaved;
+    processingStreamDescription.mFormatFlags =
+            kAudioFormatFlagsNativeEndian | kAudioFormatFlagIsSignedInteger | kAudioFormatFlagIsNonInterleaved;
@@ -305 +304,0 @@ void errorCallback(const FLAC__StreamDecoder *decoder, FLAC__StreamDecoderErrorS
-

Have any feedback or feature suggestions? Share it here.

[Copilot]

Pull request overview

Adjusts the FLAC decoder’s Linear PCM AudioStreamBasicDescription flags so kAudioFormatFlagIsAlignedHigh is only set when the decoded samples are actually stored high-aligned in a 32-bit container, matching the decoder’s sample-shifting behavior.

Changes:

• Stop unconditionally setting kAudioFormatFlagIsAlignedHigh for FLAC processing output.
• Conditionally add kAudioFormatFlagIsAlignedHigh only when _streamInfo.bits_per_sample != 32.

---

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

outdated suggestion