[polly] [clang] [clang-format] Add AllowShortType option for AlwaysBreakAfterReturnType. (PR #78011)
    Owen Pan via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Sun Jan 28 14:25:40 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) &&
----------------
owenca wrote:
What I suggested is more consistent with the existing code in clang-format. Also, explicitly including/excluding every enum value is cumbersome and doesn't provide any real advantage, IMO.
https://github.com/llvm/llvm-project/pull/78011
    
    
More information about the cfe-commits
mailing list