dryark/strawberry
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

TagReaderGME: Fix use of Qt::CaseInsensitive and length check

Browse Source
This commit is contained in:
Jonas Kvinge
2025-02-08 01:17:44 +01:00
parent b17cae6ec7
commit 215627b0e4
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/tagreader/tagreadergme.cpp
View File

@@ -39,12 +39,12 @@ using namespace Qt::Literals::StringLiterals;
#undef QStringToTString #undef QStringToTString
bool GME::IsSupportedFormat(const QFileInfo &fileinfo) { bool GME::IsSupportedFormat(const QFileInfo &fileinfo) {
return fileinfo.exists() && (fileinfo.completeSuffix().endsWith("spc"_L1, Qt::CaseInsensitive) || fileinfo.completeSuffix().endsWith("vgm"_L1), Qt::CaseInsensitive); return fileinfo.exists() && (fileinfo.completeSuffix().endsWith("spc"_L1, Qt::CaseInsensitive) || fileinfo.completeSuffix().endsWith("vgm"_L1, Qt::CaseInsensitive));
} }
TagReaderResult GME::ReadFile(const QFileInfo &fileinfo, Song *song) { TagReaderResult GME::ReadFile(const QFileInfo &fileinfo, Song *song) {
if (fileinfo.completeSuffix().endsWith("spc"_L1), Qt::CaseInsensitive) { if (fileinfo.completeSuffix().endsWith("spc"_L1, Qt::CaseInsensitive)) {
return SPC::Read(fileinfo, song); return SPC::Read(fileinfo, song);
} }
if (fileinfo.completeSuffix().endsWith("vgm"_L1, Qt::CaseInsensitive)) { if (fileinfo.completeSuffix().endsWith("vgm"_L1, Qt::CaseInsensitive)) {
@@ -106,7 +106,7 @@ TagReaderResult GME::SPC::Read(const QFileInfo &fileinfo, Song *song) {
if (length_bytes.size() >= INTRO_LENGTH_SIZE) { if (length_bytes.size() >= INTRO_LENGTH_SIZE) {
quint64 length_in_sec = ConvertSPCStringToNum(length_bytes); quint64 length_in_sec = ConvertSPCStringToNum(length_bytes);
if (!length_in_sec || length_in_sec >= 0x1FFF) { if (length_in_sec <= 0 || length_in_sec >= 0x1FFF) {
// This means that parsing the length as a string failed, so get value LE. // This means that parsing the length as a string failed, so get value LE.
length_in_sec = length_bytes[0] | (length_bytes[1] << 8) | (length_bytes[2] << 16); length_in_sec = length_bytes[0] | (length_bytes[1] << 8) | (length_bytes[2] << 16);
} }