search

llvm / llvm-project

The LLVM Project is a collection of modular and reusable compiler and toolchain technologies.
http://llvm.org
Other
26.82k stars 10.99k forks source link

readability-identifier-naming warns on the first declaration rather than the definition #97207

Open njames93 opened 1 week ago

njames93 commented 1 week ago

Warnings for readability-identifier-naming are attached to the forward declaration rather than the definition. This can break header-filter logic when a class is forward declared and its implementation lives in a header file that we don't control. Ideally all warnings emitted by this check should be attached to the definition

https://godbolt.org/z/h74xYY9Yx

llvmbot commented 1 week ago

@llvm/issue-subscribers-clang-tidy

Author: Nathan James (njames93)

Warnings for readability-identifier-naming are attached to the forward declaration rather than the definition. This can break header-filter logic when a class is forward declared and its implementation lives in a header file that we don't control. Ideally all warnings emitted by this check should be attached to the definition https://godbolt.org/z/h74xYY9Yx