[PATCH] D113319: [clang-format] Improve require handling

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 5 14:07:52 PDT 2021


HazardyKnusperkeks added a comment.

Or am I mistaken and we want to be able to control the line break?

Also I would break before struct, class, union, but I would have needed to change the test, and I know that we at least need to talk about in in front. :)

  template<typename T>
  require X<T>
  class ...
  
  vs.
  
  template<typename T>
  require X<T> class ...



================
Comment at: clang/lib/Format/TokenAnnotator.cpp:1419
             TT_UntouchableMacroFunc, TT_ConstraintJunctions,
-            TT_StatementAttributeLikeMacro))
+            TT_StatementAttributeLikeMacro, TT_RequiresClause,
+            TT_RequiresExpression))
----------------
This one has cost me **a lot** of time, should we rethink this?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D113319/new/

https://reviews.llvm.org/D113319



More information about the cfe-commits mailing list