clang-tidy：如何抑制C头文件中的C警告？

Question

我有一个.h文件，该文件包含在C和C源文件中。其内容被包装在

#ifdef __cplusplus
extern "C" {
#endif

...

#ifdef __cplusplus
}
#endif

然而，当我将它包含在. cpp文件中时，clang-tidy会发出特定于C的消息，比如

< li >警告:在C #中包含“stdbool.h”没有任何效果；请考虑删除它[hicpp-deprecated-headers，genesize-deprecated-headers] < li >警告:包含不推荐使用的C头文件“stdlib . h”；请考虑使用' cstdlib '而不是[hicpp-deprecated-headers，genesize-deprecated-headers] < li >警告:使用“using”而不是“typedef”[现代化-使用-使用]

我喜欢这些检查，我想在我整洁的配置中保持它们处于活动状态，但当然仅适用于C代码。我无法使用而不是typedef或将头文件更改为使用

有什么方法可以告诉clang-tidy将< code>extern "C"中的代码视为C而不是C，即使包含在. cpp文件中？

叮叮当当的版本是 12.0.0。




Answer 1

这些C和C标头严格来说并不等同于C。这些警告是合法的，表明代码不符合C标准。

extern"C"仅将声明的函数和变量的链接类型声明为全局，没有修改，名称以C格式使用。它不会影响使用C特性声明的对象和函数。它与语言的生态系统无关。更重要的是，如果C标头出现在这些括号内，如果声明的函数最初在C链接中，则可能会出现链接错误。

对于 C，您必须使用#ifdef __cplusplus来包含 C 替代项，否则包括 C 替代项。

而不是：

#ifdef __cplusplus
extern "C" {
#endif

#include <stdbool.h>
#include <stdlib.h>

// your declarations

#ifdef __cplusplus
}
#endif

您将拥有一个具有以下结构的标题

// headers that are universally compatible (3rd party)

#ifdef __cplusplus
#   include <cstdlib>
// other C++ headers
// Your C++ - only declarations (incompatible with C rules) 

extern "C" {
#else
#   include <stdbool.h>
#   include <stdlib.h>
// other C headers
//  C-only declarations (incompatible with C++ rules)

#endif

// your compatible declarations available for both

#ifdef __cplusplus  
}  // extern "C"
#endif

在某些情况下，由于标头包含或声明的依赖关系所需的顺序，需要调整此结构。盲目地更改您没有编写的库头中声明的链接，并且您不能保证这些使用哪种命名约定和链接类型是一种错误，可能导致ODR违规或链接失败。

typedef问题很烦人，作为一种临时措施，您可以使用NOLINT或命令行选项来抑制< code >现代化-使用-使用。一般来说，对于可移植的头文件，至少对于结构类型，这将由宏定义来完成。有一个问题，typedef和alias声明在某些情况下不能通过标准方法转换，例如，考虑函数指针类型的声明。

Answer 2

匿名用户

Clang-Tidy可以使用特殊的< code>NOLINT或< code>NOLINTNEXTLINE注释来取消对特定行的警告。它完全适用于您的使用情形:

• 有些行包含遗留的或不太好的C代码
• 这样做是有充分理由的 - 这里的代码必须可由C编译器解析。

使用它时，更高的风险是滥用它，并在可能改进编码的地方沉默警告。但是，当您需要从C和C源代码中都使用一个标题时，并且您已经仔细阅读了两次NOLINTed行，这是非常好的，至少是IMHO。此外，甚至可以指示警告保持沉默：

#ifdef __cplusplus
extern "C" {
#endif
// NOLINTNEXTLINE(hicpp-deprecated-headers,modernize-deprecated-headers) C compatible code
#include <stdbool.h>
#include <stdlib.h>     // NOLINT C code requires that header
...

#ifdef __cplusplus
}
#endif