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

I think `CHECK_PARSE_LIST` is for testing empty lists. Instead, you should add 
something like the following to line 962 above:
```
  CHECK_PARSE("BasedOnStyle: Google", Macros, std::vector<std::string>({...}));
```

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