[PATCH] D59309: [clang-format] BreakAfterReturnType ignored on functions with numeric template parameters
Manuel Klimek via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 14 06:31:01 PDT 2019
klimek added a comment.
problem is the olden C++ problem clang-format will never be able to solve, thus all these crazy heuristics.
So, inching closer to understanding this - I totally missed the isLiteral() that short-circuits out before. I'm more happy with your current approach, except I'd turn it around:
I'd only short-circuit out of the loop returning false in line 2064 if isLiteral() && (TemplateOpenerLevel > 0).
Alternatively, we could try to skip things within template <> - don't we have the closing > for an opening < in MatchingParen?
CHANGES SINCE LAST ACTION
More information about the cfe-commits