diff --git a/src/core/file_sys/ips_layer.cpp b/src/core/file_sys/ips_layer.cpp index 1df2f9dadd..9d46549bf0 100644 --- a/src/core/file_sys/ips_layer.cpp +++ b/src/core/file_sys/ips_layer.cpp @@ -120,7 +120,7 @@ std::array IPSwitchCompiler::GetBuildID() const { static IPSwitchRecord EscapeStringSequences(std::string_view sv) { IPSwitchRecord r{}; - for (auto it = sv.cbegin(); it != sv.cend(); ) { + for (auto it = sv.cbegin(); it < sv.cend(); ) { if (*it == '\\' && it + 1 < sv.cend()) { switch (it[1]) { case 'a': r.data[r.count] = '\a'; break; @@ -198,6 +198,8 @@ void IPSwitchCompiler::Parse(std::span bytes) { LOG_WARNING(Loader, "Unknown flag {}", line); break; } + } else if (patches.empty()) { + LOG_WARNING(Loader, "Invalid line not in a patch {}", line); } else { size_t offset = size_t(std::strtoul(line.data(), nullptr, 16)); offset += size_t(offset_shift); @@ -253,25 +255,30 @@ void IPSwitchCompiler::Parse(std::span bytes) { // now make a nominal preprocessed line: remove comments char quote = '\0'; auto const sline_start = p; + auto last_char = p; for (; p < sline.cend(); ) { // we dont check for "//", IPS checks for '/' only... - if ((!quote && p[0] == '/') + if (std::isspace(*p)) { + ++p; + } else if ((!quote && p[0] == '/') || (!quote && p[0] == '#')) { break; } else if (p[0] == '\"' || p[0] == '\'') { quote = (p[0] == quote) ? '\0' : p[0]; ++p; + last_char = p; } else if (p + 1 < sline.cend() && p[0] == '\\') { p += 2; + last_char = p; } else { ++p; + last_char = p; } } // now we have the preprocessed string ;) - std::string_view pp_str(sline_start, p); - if (pp_str.size() > 0 && !parse_line(pp_str)) { + std::string_view const pp_str(sline_start, last_char); + if (pp_str.size() > 0 && !parse_line(pp_str)) break; - } } } }