<div dir="ltr"><div dir="ltr"><br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">...<br>
<br>
1. Some options are failing for me [1], I don't know if it requires a<br>
specific version of clang-format<br></blockquote><div>I fixed these errors, and the clang-format version should be 17. I will send them after some discussions, as I've shared below.<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
2. Current options are not fully aligned with coding convention, it is<br>
easier to see what is not compatible by running the tool on an existing<br>
file [2].<br>
we need to fix them.<br><br></blockquote><div>There are some cases that we need to discuss. First of all, I applied the latest clang format that I created.<br>- In default cases for clang-format, include headers are sorted alphabetically. It improves the readability of code, and it breaks the codebase. These changes do not affect code maintenance because they are only headers.<br>- Second question about the column width. In the definition of .editorconfig, the column width is set as 100, but the previous commits violate this. Here is the sample code:<br> struct rte_eth_dev rte_eth_devices[RTE_MAX_ETHPORTS];<br>@@ -61,8 +61,7 @@ static const struct rte_eth_xstats_name_off eth_dev_stats_strings[] = {<br>        {"rx_missed_errors", offsetof(struct rte_eth_stats, imissed)},<br>        {"rx_errors", offsetof(struct rte_eth_stats, ierrors)},<br>        {"tx_errors", offsetof(struct rte_eth_stats, oerrors)},<br>-       {"rx_mbuf_allocation_errors", offsetof(struct rte_eth_stats,<br>-               rx_nombuf)},<br>+       {"rx_mbuf_allocation_errors", offsetof(struct rte_eth_stats, rx_nombuf)},<br> };<br><br>We could use this clang-format file only for the new patches and newly inserted lines. I am using VSCode IDE for development, and after some modification, I can select the modified lines and apply formatting. It is very useful for these cases. Otherwise, we need to handle so many things.<br><br></div><div>By the way, I am still working on macros. If you have any comments on this, I would be very happy to discuss it.<br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>...</blockquote></div></div>