主页 / coding / 问题 / 79076887
Accepted
alfC
Asked: 2024-10-11 13:15:55 +0800 CST

如何修复 clang-tidy 中的“多级隐式指针转换”警告？

  • 772

我有一些遗留代码，如下所示：

bool has_attribute(int kvidx) const {
    void* v = nullptr;
    int flag = 0;
    MPI_Comm_get_attr(impl_, kvidx, &v, &flag);
    return flag != 0;
}

的第三个参数MPI_Comm_get_attr 采用void**。

更正：... 它需要，但根据函数文档，void*它在语义上是。这是警告的真正来源。void**

此代码现在发出 clang-tidy 警告bugprone-multi-level-implicit-pointer-conversion。

https://releases.llvm.org/18.1.0/tools/clang/tools/extra/docs/clang-tidy/checks/bugprone/multi-level-implicit-pointer-conversion.html

Clang-tidy 建议：“从‘char **’到‘void *’的多级指针转换，请使用显式转换”

我该如何应用这种“显式转换”？

我尝试过，MPI_Comm_get_attr(impl_, kvidx, static_cast<void**>(&v), &flag);但是 clang 仍然在抱怨。

需要明确的是，据我所知该代码是正确的，我正在尝试遵循 clang-tidy 的建议（不抑制警告）

c++

1 个回答

  1. Best Answer

    从它的观点来看，Clang-tidy 是正确的。

    该问题源于 API 的行为。

    根据MPI_Comm_get_attr文档，这是该函数的签名：

    int MPI_Comm_get_attr(MPI_Comm comm，int comm_keyval，void *attribute_val，int *flag)

    第三个参数是 a void*，而不是 a void**，因此 Clang-tidy 正确检测到需要进行多级转换。

    然而，MPI_Comm_get_attr文档还指出：

    尽管 attr_value 参数被声明为 void *，但它实际上是 void 指针的地址。有关更多详细信息，请参阅标准中的理由。

    因此，你提出的&v论点是正确的。

    我看到的唯一解决方案是抑制这部分代码的 Clang-tidy。

    • 3

相关问题