From 8fcda2daa23fc7635d08b8d02abb1103c8ec7dfd Mon Sep 17 00:00:00 2001 From: Ryan Francesconi Date: Sun, 17 May 2026 07:18:41 -0700 Subject: [PATCH] [FLAC] Pack hasiXML/hasBEXT with scanned to save padding Per review feedback on #1364: moving the two new bool flags to the end of FilePrivate, next to the existing `scanned` bool, lets the compiler coalesce the three bytes into the same trailing padding slot. Saves one machine word per FLAC::File instance versus placing the flags mid-struct between bextData and the List. Pure layout change, no behaviour difference. Test suite still green. --- taglib/flac/flacfile.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/taglib/flac/flacfile.cpp b/taglib/flac/flacfile.cpp index 99acb62e..91bb5897 100644 --- a/taglib/flac/flacfile.cpp +++ b/taglib/flac/flacfile.cpp @@ -72,13 +72,13 @@ public: ByteVector xiphCommentData; String iXMLData; ByteVector bextData; - bool hasiXML { false }; - bool hasBEXT { false }; List blocks; offset_t flacStart { 0 }; offset_t streamStart { 0 }; bool scanned { false }; + bool hasiXML { false }; + bool hasBEXT { false }; }; ////////////////////////////////////////////////////////////////////////////////