[clang] [Format] Configure ASSIGN_OR_RETURN macros for Google style (PR #169037)

Ilya Biryukov via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 9 02:05:49 PST 2026


================
@@ -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:

Done. I've tried to keep it similar to other tests that have the defaults and to follow your suggestions.
I also introduced a simple helper to allow passing a matcher rather than a specific value (I did not want to include the whole contents of the macro definition, just the prefix of it; it looks clearer).


Let me know if this matches what you wanted, my intuition wrt to how such tests should look like is still developing.

https://github.com/llvm/llvm-project/pull/169037


More information about the cfe-commits mailing list