Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[clang-tidy] enable cppcoreguidelines-pro-type-cstyle-cast check #55459

Closed
wants to merge 36 commits into from

Conversation

RedContritio
Copy link
Contributor

@RedContritio RedContritio commented Jul 16, 2023

PR types

Others

PR changes

Others

Description

初步移除了约 4055 个 cppcoreguidelines-pro-type-cstyle-cast 警告。

自己的日志部分

  1. 编译 Paddle
cmake ..  -DWITH_GPU=ON -DPY_VERSION=3.7 -DCMAKE_EXPORT_COMPILE_COMMANDS=ON -DWITH_NCCL=ON

make -j$(nproc)

@paddle-bot
Copy link

paddle-bot bot commented Jul 16, 2023

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added contributor External developers status: proposed labels Jul 16, 2023
@luotao1 luotao1 self-assigned this Jul 17, 2023
@RedContritio RedContritio force-pushed the clang_tidy_04 branch 7 times, most recently from 82d7b79 to 3a99755 Compare July 22, 2023 14:32
…l.cc to remove 4 cppcoreguidelines-pro-type-cstyle-cast warnings
…/interceptor_ping_pong_with_brpc_test.cc to remove 2 cppcoreguidelines-pro-type-cstyle-cast warnings
…e 122 cppcoreguidelines-pro-type-cstyle-cast warnings
…emove 23 cppcoreguidelines-pro-type-cstyle-cast warnings
…ator/python_c_gen.py to remove 6 cppcoreguidelines-pro-type-cstyle-cast warnings
…coreguidelines-pro-type-cstyle-cast warnings
…move 40 cppcoreguidelines-pro-type-cstyle-cast warnings
…hreads_tester.cc to remove 1 cppcoreguidelines-pro-type-cstyle-cast warning
…_generator.cc to remove 1496 cppcoreguidelines-pro-type-cstyle-cast warnings
…client.cc to remove 6 cppcoreguidelines-pro-type-cstyle-cast warnings
…ove 4 cppcoreguidelines-pro-type-cstyle-cast warnings
…ibrator.cc to remove 1 cppcoreguidelines-pro-type-cstyle-cast warning
…server.cc to remove 11 cppcoreguidelines-pro-type-cstyle-cast warnings
…ls.cc to remove 1 cppcoreguidelines-pro-type-cstyle-cast warning
…ient.cc to remove 1 cppcoreguidelines-pro-type-cstyle-cast warning
… remove 3 cppcoreguidelines-pro-type-cstyle-cast warnings
… to remove 1 cppcoreguidelines-pro-type-cstyle-cast warning
… to remove 1 cppcoreguidelines-pro-type-cstyle-cast warning
…ator/python_c_gen.py to remove 940 cppcoreguidelines-pro-type-cstyle-cast warnings
…remove 1 cppcoreguidelines-pro-type-cstyle-cast warning
…o remove 42 cppcoreguidelines-pro-type-cstyle-cast warnings
… to remove 1 cppcoreguidelines-pro-type-cstyle-cast warning
…to remove 3 cppcoreguidelines-pro-type-cstyle-cast warnings
…ove 7 cppcoreguidelines-pro-type-cstyle-cast warnings
… to remove 2 cppcoreguidelines-pro-type-cstyle-cast warnings
…ppcoreguidelines-pro-type-cstyle-cast warnings
…remove 12 cppcoreguidelines-pro-type-cstyle-cast warnings
…pcoreguidelines-pro-type-cstyle-cast warnings
… 1 cppcoreguidelines-pro-type-cstyle-cast warning
@RedContritio RedContritio changed the title 【测试】[clang-tidy] enable cppcoreguidelines-pro-type-cstyle-cast check [clang-tidy] enable cppcoreguidelines-pro-type-cstyle-cast check Jul 23, 2023
@paddle-ci-bot
Copy link

paddle-ci-bot bot commented Jul 31, 2023

Sorry to inform you that cb369ce's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
contributor External developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants