Re: [PATCH] devtools: add .clang-format file

2024-05-04 Thread Abdullah Ömer Yamaç
Hello, I am facing a problem while creating the configuration. I am trying to make a clang-format that formats codes without changing any codebases. Here is the question: clang-format related config: ... BraceWrapping: AfterFunction: true ... If the configuration supports braces after fun

Re: [PATCH] devtools: add .clang-format file

2024-04-30 Thread Abdullah Ömer Yamaç
> ... > > 1. Some options are failing for me [1], I don't know if it requires a > specific version of clang-format > I fixed these errors, and the clang-format version should be 17. I will send them after some discussions, as I've shared below. > > 2. Current options are not fully aligned with cod

Re: [PATCH] devtools: add .clang-format file

2024-04-29 Thread Stephen Hemminger
On Mon, 29 Apr 2024 14:32:43 +0100 Ferruh Yigit wrote: > On 4/29/2024 2:04 PM, Abdullah Ömer Yamaç wrote: > > clang-format is a tool to format C/C++/Objective-C code. It can be used > > to reformat code to match a given coding style, or to ensure that code > > adheres to a specific coding style.

Re: [PATCH] devtools: add .clang-format file

2024-04-29 Thread Stephen Hemminger
On Mon, 29 Apr 2024 14:32:43 +0100 Ferruh Yigit wrote: > ``` > RTE_ETH_FOREACH_DEV(p) I think all those foreach macros need to be in clang format. For example the kernel clang format has: # Taken from: # git grep -h '^#define [^[:space:]]*for_each[^[:space:]]*(' include/ tools/ \ #

Re: [PATCH] devtools: add .clang-format file

2024-04-29 Thread Ferruh Yigit
On 4/29/2024 2:04 PM, Abdullah Ömer Yamaç wrote: > clang-format is a tool to format C/C++/Objective-C code. It can be used > to reformat code to match a given coding style, or to ensure that code > adheres to a specific coding style. It helps to maintain a consistent > coding style across the DPDK