zone/OpenAssetTools
zone/OpenAssetTools
2
0
Fork 0
mirror of https://github.com/Laupetin/OpenAssetTools.git synced 2025-04-19 15:52:53 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix broken stringization resolution

Browse Source
This commit is contained in:
Jan 2023-12-30 13:52:05 +01:00
parent 3901ea2d3c
commit 2af58e5cac
No known key found for this signature in database
GPG Key ID: 44B581F78FF5C57C
1 changed files with 11 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
src/Parser/Parsing/Impl/DefinesStreamProxy.cpp
View File

@ -69,10 +69,11 @@ void DefinesStreamProxy::Define::IdentifyParameters(const std::vector<std::strin
(wordStart > 0 && m_value[wordStart - 1] == '#') (wordStart > 0 && m_value[wordStart - 1] == '#')
// Make sure it's not a token pasting operator // Make sure it's not a token pasting operator
&& (wordStart <= 1 || m_value[wordStart - 2] != '#'); && (wordStart <= 1 || m_value[wordStart - 2] != '#');
const auto stringizeOffset = stringize ? 1 : 0;
m_value.erase(wordStart, i - wordStart); m_value.erase(wordStart - stringizeOffset, i - wordStart + stringizeOffset);
m_parameter_positions.emplace_back(parameterIndex, wordStart, stringize); m_parameter_positions.emplace_back(parameterIndex, wordStart - stringizeOffset, stringize);
i = wordStart; i = wordStart - stringizeOffset;
} }
inWord = false; inWord = false;
@ -106,9 +107,10 @@ void DefinesStreamProxy::Define::IdentifyParameters(const std::vector<std::strin
(wordStart > 0 && m_value[wordStart - 1] == '#') (wordStart > 0 && m_value[wordStart - 1] == '#')
// Make sure it's not a token pasting operator // Make sure it's not a token pasting operator
&& (wordStart <= 1 || m_value[wordStart - 2] != '#'); && (wordStart <= 1 || m_value[wordStart - 2] != '#');
const auto stringizeOffset = stringize ? 1 : 0;
m_value.erase(wordStart, m_value.size() - wordStart); m_value.erase(wordStart - stringizeOffset, m_value.size() - wordStart + stringizeOffset);
m_parameter_positions.emplace_back(parameterIndex, wordStart, stringize); m_parameter_positions.emplace_back(parameterIndex, wordStart - stringizeOffset, stringize);
} }
} }
} }
@ -607,9 +609,8 @@ void DefinesStreamProxy::InsertMacroParameters(std::ostringstream& out, const De
auto lastPos = 0u; auto lastPos = 0u;
for (const auto& parameterPosition : macro->m_parameter_positions) for (const auto& parameterPosition : macro->m_parameter_positions)
{ {
const auto stringizeSkipLastChar = parameterPosition.m_stringize ? 1 : 0;
if (lastPos < parameterPosition.m_parameter_position) if (lastPos < parameterPosition.m_parameter_position)
out << std::string(macro->m_value, lastPos, parameterPosition.m_parameter_position - lastPos - stringizeSkipLastChar); out << std::string(macro->m_value, lastPos, parameterPosition.m_parameter_position - lastPos);
if (parameterPosition.m_parameter_index < parameterValues.size()) if (parameterPosition.m_parameter_index < parameterValues.size())
{ {
@ -647,8 +648,8 @@ void DefinesStreamProxy::ExpandMacro(ParserLine& line,
void DefinesStreamProxy::ContinueMacroParameters( void DefinesStreamProxy::ContinueMacroParameters(
const ParserLine& line, unsigned& linePos, MacroParameterState& state, const std::string& input, unsigned& inputPos) const ParserLine& line, unsigned& linePos, MacroParameterState& state, const std::string& input, unsigned& inputPos)
{ {
const auto lineLength = line.m_line.size(); const auto inputLength = input.size();
while (state.m_parameter_state != ParameterState::NOT_IN_PARAMETERS && inputPos < lineLength) while (state.m_parameter_state != ParameterState::NOT_IN_PARAMETERS && inputPos < inputLength)
{ {
const auto c = input[inputPos]; const auto c = input[inputPos];
@ -773,7 +774,7 @@ void DefinesStreamProxy::ProcessNestedMacros(ParserLine& line, unsigned& linePos
callstack.push_back(nestedMacro); callstack.push_back(nestedMacro);
MacroParameterState nestedMacroState; MacroParameterState nestedMacroState;
ExtractParametersFromMacroUsage(line, pos, nestedMacroState, input, inputPos); ExtractParametersFromMacroUsage(line, linePos, nestedMacroState, input, pos);
if (nestedMacroState.m_parameter_state != ParameterState::NOT_IN_PARAMETERS) if (nestedMacroState.m_parameter_state != ParameterState::NOT_IN_PARAMETERS)
throw ParsingException(CreatePos(line, linePos), "Unbalanced brackets in macro parameters"); throw ParsingException(CreatePos(line, linePos), "Unbalanced brackets in macro parameters");
ExpandMacro(line, linePos, ss, callstack, nestedMacro, nestedMacroState.m_parameters); ExpandMacro(line, linePos, ss, callstack, nestedMacro, nestedMacroState.m_parameters);