Skip to content

Commit dd99b1c

Browse files
JohnCoconutminggo
authored andcommitted
add clang-tidy-bugprone options (#19750)
* add clang-tidy-bugprone options * bugprone-argument-comment * bugprone-bool-pointer-implicit-conversion * bugprone-dangling-handle * bugprone-fold-init-type * bugprone-forward-declaration-namespace * enable more clang-tidy bugprone options * bugprone-forwarding-reference-overload * bugprone-inaccurate-erase, * bugprone-incorrect-roundings,
1 parent ace5d96 commit dd99b1c

File tree

3 files changed

+13
-3
lines changed

3 files changed

+13
-3
lines changed

Diff for: .clang-tidy

+8
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,14 @@ Checks: >
1313
readability-redundant-control-flow,
1414
readability-redundant-string-cstr,
1515
readability-string-compare,
16+
bugprone-argument-comment,
17+
bugprone-bool-pointer-implicit-conversion,
18+
bugprone-dangling-handle,
19+
bugprone-fold-init-type,
20+
bugprone-forward-declaration-namespace,
21+
bugprone-forwarding-reference-overload,
22+
bugprone-inaccurate-erase,
23+
bugprone-incorrect-roundings,
1624
1725
WarningsAsErrors: '*'
1826
HeaderFilterRegex: '/(?!external)/.*'

Diff for: cocos/platform/linux/CCDevice-linux.cpp

+2-1
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ THE SOFTWARE.
3434
#include <stdio.h>
3535

3636
#include <algorithm>
37+
#include <cmath>
3738
#include <vector>
3839
#include <map>
3940
#include <string>
@@ -98,7 +99,7 @@ int Device::getDPI()
9899
*/
99100
double xres = ((((double) DisplayWidth(dpy,scr)) * 25.4) /
100101
((double) DisplayWidthMM(dpy,scr)));
101-
dpi = (int) (xres + 0.5);
102+
dpi = std::lround(xres);
102103
//printf("dpi = %d\n", dpi);
103104
XCloseDisplay (dpy);
104105
}

Diff for: tests/cpp-tests/Classes/BillBoardTest/BillBoardTest.cpp

+3-2
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
#include "3d/CCBillBoard.h"
2929

3030
#include <algorithm>
31+
#include <cmath>
3132
#include "../testResource.h"
3233

3334
USING_NS_CC;
@@ -134,7 +135,7 @@ BillBoardTest::BillBoardTest()
134135
for (unsigned int i = 0; i < 4; ++i)
135136
{
136137
Layer *layer = Layer::create();
137-
auto billboard = BillBoard::create(imgs[(unsigned int)(CCRANDOM_0_1() * 1 + 0.5)]);
138+
auto billboard = BillBoard::create(imgs[(unsigned int)(std::lround(CCRANDOM_0_1()))]);
138139
billboard->setScale(0.5f);
139140
billboard->setPosition3D(Vec3(0.0f, 0.0f, CCRANDOM_MINUS1_1() * 150.0f));
140141
billboard->setOpacity(CCRANDOM_0_1() * 128 + 128);
@@ -238,7 +239,7 @@ void BillBoardTest::addNewBillBoardWithCoords(Vec3 p)
238239
std::string imgs[3] = {"Images/Icon.png", "Images/r2.png"};
239240
for (unsigned int i = 0; i < 10; ++i)
240241
{
241-
auto billboard = BillBoard::create(imgs[(unsigned int)(CCRANDOM_0_1() * 1 + 0.5)]);
242+
auto billboard = BillBoard::create(imgs[(unsigned int)(std::lround(CCRANDOM_0_1()))]);
242243
billboard->setScale(0.5f);
243244
billboard->setPosition3D(Vec3(p.x, p.y, -150.0f + 30 * i));
244245
billboard->setOpacity(CCRANDOM_0_1() * 128 + 128);

0 commit comments

Comments
 (0)