This is an archive of the discontinued LLVM Phabricator instance.

[clang][analyzer][NFC] Change PlacementNewChecker into PlacementNew in documentation
ClosedPublic

Authored by PiotrZSL on Aug 11 2023, 4:09 AM.

Details

Summary

Check name according to Checkers.td is actually a PlacementNew.

Diff Detail

Event Timeline

PiotrZSL created this revision.Aug 11 2023, 4:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 11 2023, 4:09 AM
PiotrZSL requested review of this revision.Aug 11 2023, 4:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 11 2023, 4:09 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
PiotrZSL updated this revision to Diff 549336.Aug 11 2023, 4:12 AM

Micro fix ("" characters).

steakhal accepted this revision.Aug 11 2023, 8:16 AM

Correct. Thanks.

This revision is now accepted and ready to land.Aug 11 2023, 8:16 AM

Do we have other typos like this?

Do we have other typos like this?

I don't think so, only mess in overall documentation: https://github.com/llvm/llvm-project/issues/64614
I updated gen-static-analyzer-docs.py, and it shown me that some documentation is missing: https://clang.llvm.org/extra/clang-tidy/checks/list.html.
You can see it, as there are no links for some checks.

Do we have other typos like this?

I don't think so, only mess in overall documentation: https://github.com/llvm/llvm-project/issues/64614
I updated gen-static-analyzer-docs.py, and it shown me that some documentation is missing: https://clang.llvm.org/extra/clang-tidy/checks/list.html.
You can see it, as there are no links for some checks.

Yea, it's a complete mess, I agree.
I have no stakes on the docs, but I know that Ericsson folks do. They might wanna chim in. @donat.nagy @Szelethus