[Bf-blender-cvs] [d7ac659e023] master: Cleanup: Clang tidy
Hans Goudey
noreply at git.blender.org
Thu Jan 27 17:54:16 CET 2022
Commit: d7ac659e02327ceb433bf69a624d76fa7c65b129
Author: Hans Goudey
Date: Thu Jan 27 10:53:34 2022 -0600
Branches: master
https://developer.blender.org/rBd7ac659e02327ceb433bf69a624d76fa7c65b129
Cleanup: Clang tidy
Use nullptr, use named parameters, fix deprecated header
===================================================================
M source/blender/draw/engines/workbench/workbench_shader.cc
M source/blender/render/intern/texture_margin.cc
===================================================================
diff --git a/source/blender/draw/engines/workbench/workbench_shader.cc b/source/blender/draw/engines/workbench/workbench_shader.cc
index 011a3fd3b13..bbc0bc02b03 100644
--- a/source/blender/draw/engines/workbench/workbench_shader.cc
+++ b/source/blender/draw/engines/workbench/workbench_shader.cc
@@ -64,7 +64,7 @@ static struct {
struct GPUShader *volume_sh[2][2][3][2];
-} e_data = {{{{NULL}}}};
+} e_data = {{{{nullptr}}}};
/* -------------------------------------------------------------------- */
/** \name Conversions
diff --git a/source/blender/render/intern/texture_margin.cc b/source/blender/render/intern/texture_margin.cc
index 0e47c2cec8a..4fbd20eeaf1 100644
--- a/source/blender/render/intern/texture_margin.cc
+++ b/source/blender/render/intern/texture_margin.cc
@@ -43,7 +43,7 @@
#include "RE_texture_margin.h"
#include <algorithm>
-#include <math.h>
+#include <cmath>
#include <valarray>
namespace blender::render::texturemargin {
@@ -128,7 +128,8 @@ class TextureMarginMap {
&zspan_, this, &(v1[0]), &(v2[0]), &(v3[0]), TextureMarginMap::zscan_store_pixel);
}
- static void zscan_store_pixel(void *map, int x, int y, float, float)
+ static void zscan_store_pixel(
+ void *map, int x, int y, [[maybe_unused]] float u, [[maybe_unused]] float v)
{
/* NOTE: Not thread safe, see comment above.
*
@@ -441,17 +442,17 @@ static void generate_margin(ImBuf *ibuf,
int tottri;
MLoopTri const *looptri;
- MLoopTri *looptri_mem = NULL;
+ MLoopTri *looptri_mem = nullptr;
if (me) {
- BLI_assert(dm == NULL);
+ BLI_assert(dm == nullptr);
totpoly = me->totpoly;
totloop = me->totloop;
totedge = me->totedge;
mpoly = me->mpoly;
mloop = me->mloop;
- if ((uv_layer == NULL) || (uv_layer[0] == '\0')) {
+ if ((uv_layer == nullptr) || (uv_layer[0] == '\0')) {
mloopuv = static_cast<MLoopUV const *>(CustomData_get_layer(&me->ldata, CD_MLOOPUV));
}
else {
@@ -467,9 +468,9 @@ static void generate_margin(ImBuf *ibuf,
looptri = looptri_mem;
}
else {
- BLI_assert(dm != NULL);
- BLI_assert(me == NULL);
- BLI_assert(mloopuv == NULL);
+ BLI_assert(dm != nullptr);
+ BLI_assert(me == nullptr);
+ BLI_assert(mloopuv == nullptr);
totpoly = dm->getNumPolys(dm);
totedge = dm->getNumEdges(dm);
totloop = dm->getNumLoops(dm);
@@ -510,7 +511,7 @@ static void generate_margin(ImBuf *ibuf,
}
BLI_assert(lt->poly < 0x80000000); // NOTE: we need the top bit for the dijkstra distance map
- map.rasterize_tri(vec[0], vec[1], vec[2], lt->poly, draw_new_mask ? mask : NULL);
+ map.rasterize_tri(vec[0], vec[1], vec[2], lt->poly, draw_new_mask ? mask : nullptr);
}
char *tmpmask = (char *)MEM_dupallocN(mask);
@@ -543,7 +544,7 @@ static void generate_margin(ImBuf *ibuf,
void RE_generate_texturemargin_adjacentfaces(
ImBuf *ibuf, char *mask, const int margin, const Mesh *me, char const *uv_layer)
{
- blender::render::texturemargin::generate_margin(ibuf, mask, margin, me, NULL, uv_layer);
+ blender::render::texturemargin::generate_margin(ibuf, mask, margin, me, nullptr, uv_layer);
}
void RE_generate_texturemargin_adjacentfaces_dm(ImBuf *ibuf,
@@ -551,5 +552,5 @@ void RE_generate_texturemargin_adjacentfaces_dm(ImBuf *ibuf,
const int margin,
DerivedMesh *dm)
{
- blender::render::texturemargin::generate_margin(ibuf, mask, margin, NULL, dm, NULL);
+ blender::render::texturemargin::generate_margin(ibuf, mask, margin, nullptr, dm, nullptr);
}
More information about the Bf-blender-cvs
mailing list