Conversation
|
Okay, but why? Contextless changes are unlikely to get merged. |
@bnoordhuis It has warnings about the deprecated header, and in that file, all other header included is cpp style with no The comment change is because it was |
|
Can you please consider sharing links to code, rather than screenshots? Screenshots can be harder to read (for folks using screen readers, or if the color scheme you're using is not accessible to some visual impairments), and consume more data to download. Links are also better at giving context. |
|
But I cannot show the warning without screenshots |
|
a link to the clang-tidy rule would be helpful |
Added |



Ref: https://releases.llvm.org/13.0.0/tools/clang/tools/extra/docs/clang-tidy/checks/modernize-deprecated-headers.html