[PATCH 0/6] fix the curly braces mismatch problem
Bruce Richardson
bruce.richardson at intel.com
Mon Feb 7 10:29:58 CET 2022
On Sun, Feb 06, 2022 at 08:44:08AM +0800, Weiguo Li wrote:
> To make C header file compatible with C++ linking, a conventional
> practice enclose the code with braces as below:
>
> #ifdef __cplusplus extern "C" { #endif
>
> ...
>
> #ifdef __cplusplus } #endif
>
> Some alternative forms of "#ifdef __cplusplus", like "#if
> defined(__cplusplus)" or "# ifdef __cplusplus", also work.
>
> While it's suspicious when a header file matchs this regular expression
> "if.*__cplusplus" exactly once.
>
> The following script is used to find these files: grep -r ~/git/dpdk
> --include=*.h -e "if.*__cplusplus" -c|grep ":1$"
>
> Apart from two false positive cases, I found some real issues -- some
> missing the left brace parts, the others missing the right brace parts.
>
> In one of the cases, the c++ guard is removed since it's useless in
> private header as previous patch (http://dpdk.org/patch/98948) noted.
> For the other cases, the missing parts are supplemented.
>
Any headers that have filenames not starting with "rte_" are internal
headers, and so should have the guards removed. Therefore, I think that
most patches in this set should just be removing guards. From my tests with
building our public headers in C++ files, resulting in series[1], I only
found one public header with an incorrect guard.
/Bruce
[1]http://patches.dpdk.org/project/dpdk/list/?series=21466
More information about the dev
mailing list