owenca wrote:

> @owenca, just to confirm - you are suggesting to add `Custom` string to 
> `AllowShortFunctionsOnASingleLine` and new parameter 
> (`AllowShortFunctionsOnASingleLineOptions` for example) which will control 
> behavior if `AllowShortFunctionsOnASingleLine == Custom`?

Yep.

> What options should be present? I think we can start with `Inline`, `Empty`, 
> `static inline`, `constexpr` (?).

Yes, but can you split it into two patches? The first would be NFC and only 
cover `Inline` and `Empty`.

https://github.com/llvm/llvm-project/pull/133598
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to