Mirrorlandia_minetest/minetest
1
0
Fork 1
mirror of https://github.com/minetest/minetest.git synced 2024-11-23 16:13:46 +01:00
Code Issues Packages Projects Releases Wiki Activity

[clang-tidy] Promote some performance-* as a coding error (#7194)

Browse Source 
* Promote performance-type-promotion-in-math-fn as a coding error
* Promote performance-faster-string-find too (which is not problematic currently)
* Same for performance-implicit-cast-in-loop
* Fix remaining tidy points
This commit is contained in:
Loïc Blot 2018-04-04 20:49:42 +02:00 committed by GitHub
parent 392e80e3f1
commit 077f231111
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 8 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/content_cao.cpp
View File

@ -908,8 +908,7 @@ void GenericCAO::step(float dtime, ClientEnvironment *env)
updateTextures(m_previous_texture_modifier); updateTextures(m_previous_texture_modifier);
} }
} }
if(!getParent() && fabs(m_prop.automatic_rotate) > 0.001) if (!getParent() && std::fabs(m_prop.automatic_rotate) > 0.001) {
{
m_yaw += dtime * m_prop.automatic_rotate * 180 / M_PI; m_yaw += dtime * m_prop.automatic_rotate * 180 / M_PI;
updateNodePos(); updateNodePos();
} }

12
src/game.cpp
View File

@ -3638,12 +3638,12 @@ void Game::updateFrame(ProfilerGraph *graph, RunStats *stats, f32 dtime,
float time_of_day_smooth = runData.time_of_day_smooth; float time_of_day_smooth = runData.time_of_day_smooth;
float time_of_day = client->getEnv().getTimeOfDayF(); float time_of_day = client->getEnv().getTimeOfDayF();
static const float maxsm = 0.05; static const float maxsm = 0.05f;
static const float todsm = 0.05; static const float todsm = 0.05f;
if (fabs(time_of_day - time_of_day_smooth) > maxsm && if (std::fabs(time_of_day - time_of_day_smooth) > maxsm &&
fabs(time_of_day - time_of_day_smooth + 1.0) > maxsm && std::fabs(time_of_day - time_of_day_smooth + 1.0) > maxsm &&
fabs(time_of_day - time_of_day_smooth - 1.0) > maxsm) std::fabs(time_of_day - time_of_day_smooth - 1.0) > maxsm)
time_of_day_smooth = time_of_day; time_of_day_smooth = time_of_day;
if (time_of_day_smooth > 0.8 && time_of_day < 0.2) if (time_of_day_smooth > 0.8 && time_of_day < 0.2)
@ -3715,7 +3715,7 @@ void Game::updateFrame(ProfilerGraph *graph, RunStats *stats, f32 dtime,
video::EFT_FOG_LINEAR, video::EFT_FOG_LINEAR,
100000 * BS, 100000 * BS,
110000 * BS, 110000 * BS,
0.01, 0.01f,
false, // pixel fog false, // pixel fog
false // range fog false // range fog
); );

2
util/travis/clangtidy.sh
View File

@ -19,7 +19,7 @@ cd ..
echo "Performing clang-tidy checks..." echo "Performing clang-tidy checks..."
./util/travis/run-clang-tidy.py -clang-tidy-binary=${CLANG_TIDY} -p cmakebuild \ ./util/travis/run-clang-tidy.py -clang-tidy-binary=${CLANG_TIDY} -p cmakebuild \
-checks='-*,modernize-use-emplace,modernize-avoid-bind,performance-*' \ -checks='-*,modernize-use-emplace,modernize-avoid-bind,performance-*' \
-warningsaserrors='-*,modernize-use-emplace' \ -warningsaserrors='-*,modernize-use-emplace,performance-type-promotion-in-math-fn,performance-faster-string-find,performance-implicit-cast-in-loop' \
-no-command-on-stdout -quiet \ -no-command-on-stdout -quiet \
files 'src/.*' files 'src/.*'
RET=$? RET=$?