================
@@ -967,6 +967,7 @@ TEST(ConfigParseTest, ParsesConfiguration) {
               std::vector<std::string>({"QUNUSED", "QT_REQUIRE_VERSION"}));
 
   CHECK_PARSE_LIST(JavaImportGroups);
+  Style.Macros.clear();
----------------
ilya-biryukov wrote:

`CHECK_PARSE_LIST(Macros)` would fail otherwise because we'll get the 
definitions for those `ASSIGN_OR_RETURN` macros added to the parsed file.

This looks intended, `parseConfiguration` takes the default value of the list 
from the style first and then appends whatever's written in the actual config.
I believe many of the other `.clear()` calls in the file serve the same purpose.

https://github.com/llvm/llvm-project/pull/169037
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to