Improve FLAC channel count validation

[sbooth]

No description provided.

[github-actions]

Cpp-linter Review

Used clang-format v22.1.7

Click here for the full clang-format patch

diff --git a/Sources/CSFBAudioEngine/Decoders/SFBFLACDecoder.mm b/Sources/CSFBAudioEngine/Decoders/SFBFLACDecoder.mm
index 2e370c6..c9cf23c 100644
--- a/Sources/CSFBAudioEngine/Decoders/SFBFLACDecoder.mm
+++ b/Sources/CSFBAudioEngine/Decoders/SFBFLACDecoder.mm
@@ -596,3 +596,4 @@ void errorCallback(const FLAC__StreamDecoder *decoder, FLAC__StreamDecoderErrorS
-                       unsupportedFormatError:NSLocalizedString(@"FLAC", @"")
-                       recoverySuggestion:NSLocalizedString(@"Channel count mismatch between STREAMINFO and first frame header.",
-                                                            @"")];
+                unsupportedFormatError:NSLocalizedString(@"FLAC", @"")
+                    recoverySuggestion:NSLocalizedString(
+                                               @"Channel count mismatch between STREAMINFO and first frame header.",
+                                               @"")];

Have any feedback or feature suggestions? Share it here.

[Copilot]

Pull request overview

This PR refines FLAC channel-count validation in SFBFLACDecoder by moving the STREAMINFO vs frame-header channel check to only trigger on the first decoded frame, aiming to avoid repeated per-frame validation while still catching invalid streams early.

Changes:

• Removed per-frame STREAMINFO vs frame-header channel validation.
• Added a STREAMINFO vs first frame-header channel validation with a more specific error message.

---

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

outdated suggestion

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.