aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorScott Wheeler <scott@directededge.com>2019-09-11 00:58:18 +0200
committerScott Wheeler <scott@directededge.com>2019-09-11 00:58:18 +0200
commit0b99cd9bac52e0aa398b46511580b2a6aba393cf (patch)
tree6770fc24473c24ac446f13431caddd32d168e618
parent4668cf0f9394e35ff5c2aa44797dcc31d6dc73d3 (diff)
Revert switch to other static size method
This was based on a misread of the header: at present there is no non-static size() method, so removing the argument makes the behavior incorrect.
-rw-r--r--taglib/mpeg/id3v2/id3v2framefactory.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/taglib/mpeg/id3v2/id3v2framefactory.cpp b/taglib/mpeg/id3v2/id3v2framefactory.cpp
index be07754a..ad598332 100644
--- a/taglib/mpeg/id3v2/id3v2framefactory.cpp
+++ b/taglib/mpeg/id3v2/id3v2framefactory.cpp
@@ -169,7 +169,7 @@ Frame *FrameFactory::createFrame(const ByteVector &origData, const Header *tagHe
if(version > 3 && (tagHeader->unsynchronisation() || header->unsynchronisation())) {
// Data lengths are not part of the encoded data, but since they are synch-safe
// integers they will be never actually encoded.
- ByteVector frameData = data.mid(header->size(), header->frameSize());
+ ByteVector frameData = data.mid(Frame::Header::size(version), header->frameSize());
frameData = SynchData::decode(frameData);
data = data.mid(0, Frame::Header::size(version)) + frameData;
}