Fix not ignoring whitespace after macro parameter comma

Laupetin · Dec 30, 2023 · 799fa37 · 799fa37
1 parent d3519c2
commit 799fa37
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/Parser/Parsing/Impl/DefinesStreamProxy.cpp b/src/Parser/Parsing/Impl/DefinesStreamProxy.cpp
@@ -685,7 +685,7 @@ void DefinesStreamProxy::ContinueMacroParameters(
                 state.m_parameters.emplace_back(state.m_current_parameter.str());
                 state.m_current_parameter.clear();
                 state.m_current_parameter.str(std::string());
-                m_parameter_state = ParameterState::AFTER_COMMA;
+                state.m_parameter_state = ParameterState::AFTER_COMMA;
             }
         }
         else if (c == '(' || c == '[' || c == '{')

diff --git a/test/ParserTests/Parsing/Impl/DefinesStreamProxyTests.cpp b/test/ParserTests/Parsing/Impl/DefinesStreamProxyTests.cpp
@@ -959,8 +959,8 @@ namespace test::parsing::impl::defines_stream_proxy
         ExpectLine(&proxy, 1, "");
         ExpectLine(&proxy, 2, "");
         ExpectLine(&proxy, 3, "");
-        ExpectLine(&proxy, 4, "TEST");
-        ExpectLine(&proxy, 5, "TESTTWO(testArg)");
+        ExpectLine(&proxy, 4, "\"TEST\"");
+        ExpectLine(&proxy, 5, "\"TESTTWO(testArg)\"");
 
         REQUIRE(proxy.Eof());
     }