mirror/openmw
1
0
Fork 0
mirror of https://github.com/OpenMW/openmw.git synced 2025-02-27 04:39:43 +00:00
Code Issues Releases Wiki Activity

ESM4: Make script local variable loading more reliable

Browse source
This commit is contained in:
Alexei Kotov 2024-03-21 18:16:11 +03:00
parent 82bc6674dc
commit 3721a69747
3 changed files with 13 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
components/esm4/loadinfo.cpp
View file

@ -41,7 +41,6 @@ void ESM4::DialogInfo::load(ESM4::Reader& reader)
mEditorId = ESM::RefId(mId).serializeText(); // FIXME: quick workaround to use existing code
ScriptLocalVariableData localVar;
bool ignore = false;
while (reader.getSubRecordHeader())
@ -125,22 +124,24 @@ void ESM4::DialogInfo::load(ESM4::Reader& reader)
break;
case ESM::fourCC("SLSD"):
{
localVar.clear();
ScriptLocalVariableData localVar;
reader.get(localVar.index);
reader.get(localVar.unknown1);
reader.get(localVar.unknown2);
reader.get(localVar.unknown3);
reader.get(localVar.type);
reader.get(localVar.unknown4);
mScript.localVarData.push_back(std::move(localVar));
// WARN: assumes SCVR will follow immediately
break;
}
case ESM::fourCC("SCVR"): // assumed always pair with SLSD
{
reader.getZString(localVar.variableName);
mScript.localVarData.push_back(localVar);
if (!mScript.localVarData.empty())
reader.getZString(mScript.localVarData.back().variableName);
else
reader.skipSubRecordData();
break;
}

12
components/esm4/loadscpt.cpp
View file

@ -36,8 +36,6 @@ void ESM4::Script::load(ESM4::Reader& reader)
mId = reader.getFormIdFromHeader();
mFlags = reader.hdr().record.flags;
ScriptLocalVariableData localVar;
while (reader.getSubRecordHeader())
{
const ESM4::SubRecordHeader& subHdr = reader.subRecordHeader();
@ -117,20 +115,24 @@ void ESM4::Script::load(ESM4::Reader& reader)
break;
case ESM::fourCC("SLSD"):
{
localVar.clear();
ScriptLocalVariableData localVar;
reader.get(localVar.index);
reader.get(localVar.unknown1);
reader.get(localVar.unknown2);
reader.get(localVar.unknown3);
reader.get(localVar.type);
reader.get(localVar.unknown4);
mScript.localVarData.push_back(std::move(localVar));
// WARN: assumes SCVR will follow immediately
break;
}
case ESM::fourCC("SCVR"): // assumed always pair with SLSD
reader.getZString(localVar.variableName);
mScript.localVarData.push_back(localVar);
if (!mScript.localVarData.empty())
reader.getZString(mScript.localVarData.back().variableName);
else
reader.skipSubRecordData();
break;
case ESM::fourCC("SCRV"):
{

7
components/esm4/script.hpp
View file

@ -365,13 +365,6 @@ namespace ESM4
std::uint32_t unknown4;
// SCVR
std::string variableName;
void clear()
{
index = 0;
type = 0;
variableName.clear();
}
};
struct ScriptDefinition