Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[clang-tidy] Move hicpp checks to other modules #183462

@zeyi2

Description

@zeyi2

Related discussion on discourse: https://discourse.llvm.org/t/rfc-regarding-the-current-status-of-hicpp-checks

The main goal is to move all implementations of checks in hicpp module into other modules. Currently we can keep the initial check as alias then completely remove them later (If we have to do so due to legal problems)

Current naming suggestion

  • hicpp-exception-baseclass -> bugprone-exception-baseclass
  • hicpp-ignored-remove-result -> bugprone-unused-return-value
  • hicpp-multiway-paths-covered -> readability-multiway-paths-covered
  • hicpp-signed-bitwise -> bugprone-signed-bitwise
  • hicpp-no-assembler -> portability-no-assembler

I opened sub-issues using these check names, but they are just my personal opinion, will update when we get a better naming alternative :)

See any sub-issue in #157287 on how to do the refactor.

Metadata

Metadata

Assignees

No one assigned

    Labels

    clang-tidymetaissueIssue to collect references to a group of similar or related issues.

    Type

    No fields configured for Task.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions