[polly] [clang] [clang-format] Add AllowShortType option for AlwaysBreakAfterReturnType. (PR #78011)

Björn Schäpers via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 29 11:55:20 PST 2024


================
@@ -587,7 +590,7 @@ bool ContinuationIndenter::mustBreak(const LineState &State) {
       !State.Line->ReturnTypeWrapped &&
       // Don't break before a C# function when no break after return type.
       (!Style.isCSharp() ||
-       Style.AlwaysBreakAfterReturnType != FormatStyle::RTBS_None) &&
+       Style.AlwaysBreakAfterReturnType > FormatStyle::RTBS_AllowShortType) &&
----------------
HazardyKnusperkeks wrote:

I know, just wanted to state my opinion. The cumbersome part could be extracted into a function. I just don't like treating enums as numbers in general. Sometimes there are exceptions, but in my opinion I would use this in clang-format only for `Standard`.

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


More information about the cfe-commits mailing list