[clang] [clang-format] Fix a bug in annotating function declaration names (PR #76206)

Owen Pan via cfe-commits cfe-commits at lists.llvm.org
Sat Mar 9 22:05:52 PST 2024


owenca wrote:

> Thanks for doing this. Does it mean we would have to add every type we use to our config? It would be much more convenient if clang-format treated the argument list the same way that it treats in definitions. For example
> 
> ```shell
> printf "int\niso_time(time_t) { return 1; }\n" | clang-format -style='{ AlwaysBreakAfterReturnType: All }'
> ```
> 
> gives
> 
> ```c
> int
> iso_time(time_t) {
>   return 1;
> }
> ```
> 
> whereas
> 
> ```shell
> printf "int\niso_time(time_t);\n" | clang-format -style='{ AlwaysBreakAfterReturnType: All }'
> ```
> 
> will need a config edit to work.

@mattmundell you don't need to add `time_t` and other common C/C++ types to `TypeNames` after 0baef3b18cdb.

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


More information about the cfe-commits mailing list