mirror of
https://github.com/taglib/taglib.git
synced 2025-07-20 05:54:20 -04:00
startsWith() should be used to avoid bugs like: s.mid(0, 4) == "Foo".
git-svn-id: svn://anonsvn.kde.org/home/kde/trunk/kdesupport/taglib@371832 283d02a7-25f6-0310-bc7c-ecb5cbfe19da
This commit is contained in:
@ -106,7 +106,7 @@ static const unsigned short sftable [4] = { 44100, 48000, 37800, 32000 };
|
||||
|
||||
void MPC::Properties::read()
|
||||
{
|
||||
if(d->data.mid(0, 3) != "MP+")
|
||||
if(!d->data.startsWith("MP+"))
|
||||
return;
|
||||
|
||||
d->version = d->data[3] & 15;
|
||||
|
@ -196,7 +196,7 @@ void ID3v1::Tag::read()
|
||||
ByteVector data = d->file->readBlock(128);
|
||||
|
||||
// some initial sanity checking
|
||||
if(data.size() == 128 && data.mid(0, 3) == "TAG")
|
||||
if(data.size() == 128 && data.startsWith("TAG"))
|
||||
parse(data);
|
||||
else
|
||||
debug("ID3v1 tag is not valid or could not be read at the specified offset.");
|
||||
|
@ -88,7 +88,7 @@ void MPEG::XingHeader::parse(const ByteVector &data)
|
||||
{
|
||||
// Check to see if a valid Xing header is available.
|
||||
|
||||
if(data.mid(0, 4) != "Xing")
|
||||
if(!data.startsWith("Xing"))
|
||||
return;
|
||||
|
||||
// If the XingHeader doesn't contain the number of frames and the total stream
|
||||
|
@ -238,7 +238,7 @@ void Ogg::PageHeader::read()
|
||||
// Sanity check -- make sure that we were in fact able to read as much data as
|
||||
// we asked for and that the page begins with "OggS".
|
||||
|
||||
if(data.size() != 27 || data.mid(0, 4) != "OggS") {
|
||||
if(data.size() != 27 || !data.startsWith("OggS")) {
|
||||
debug("Ogg::PageHeader::read() -- error reading page header");
|
||||
return;
|
||||
}
|
||||
|
Reference in New Issue
Block a user