[Bf-blender-cvs] [190170d4cc9] master: Cleanup: Clang-Tidy, readability-redundant-member-init

Sergey Sharybin noreply at git.blender.org
Fri Nov 6 11:54:59 CET 2020


Commit: 190170d4cc92ff34abe1744a10474ac4f1074086
Author: Sergey Sharybin
Date:   Fri Aug 7 15:58:58 2020 +0200
Branches: master
https://developer.blender.org/rB190170d4cc92ff34abe1744a10474ac4f1074086

Cleanup: Clang-Tidy, readability-redundant-member-init

===================================================================

M	.clang-tidy
M	source/blender/blenlib/intern/mesh_intersect.cc
M	source/blender/compositor/intern/COM_CPUDevice.cpp
M	source/blender/compositor/intern/COM_SingleThreadedOperation.cpp
M	source/blender/compositor/operations/COM_AlphaOverKeyOperation.cpp
M	source/blender/compositor/operations/COM_AlphaOverMixedOperation.cpp
M	source/blender/compositor/operations/COM_AlphaOverPremultiplyOperation.cpp
M	source/blender/compositor/operations/COM_AntiAliasOperation.cpp
M	source/blender/compositor/operations/COM_BilateralBlurOperation.cpp
M	source/blender/compositor/operations/COM_BlurBaseOperation.cpp
M	source/blender/compositor/operations/COM_BokehBlurOperation.cpp
M	source/blender/compositor/operations/COM_BokehImageOperation.cpp
M	source/blender/compositor/operations/COM_BoxMaskOperation.cpp
M	source/blender/compositor/operations/COM_BrightnessOperation.cpp
M	source/blender/compositor/operations/COM_CalculateMeanOperation.cpp
M	source/blender/compositor/operations/COM_CalculateStandardDeviationOperation.cpp
M	source/blender/compositor/operations/COM_ChangeHSVOperation.cpp
M	source/blender/compositor/operations/COM_ChannelMatteOperation.cpp
M	source/blender/compositor/operations/COM_ChromaMatteOperation.cpp
M	source/blender/compositor/operations/COM_ColorBalanceASCCDLOperation.cpp
M	source/blender/compositor/operations/COM_ColorBalanceLGGOperation.cpp
M	source/blender/compositor/operations/COM_ColorCorrectionOperation.cpp
M	source/blender/compositor/operations/COM_ColorCurveOperation.cpp
M	source/blender/compositor/operations/COM_ColorMatteOperation.cpp
M	source/blender/compositor/operations/COM_ColorRampOperation.cpp
M	source/blender/compositor/operations/COM_ColorSpillOperation.cpp
M	source/blender/compositor/operations/COM_CompositorOperation.cpp
M	source/blender/compositor/operations/COM_ConvertColorProfileOperation.cpp
M	source/blender/compositor/operations/COM_ConvertDepthToRadiusOperation.cpp
M	source/blender/compositor/operations/COM_ConvertOperation.cpp
M	source/blender/compositor/operations/COM_ConvolutionEdgeFilterOperation.cpp
M	source/blender/compositor/operations/COM_ConvolutionFilterOperation.cpp
M	source/blender/compositor/operations/COM_CropOperation.cpp
M	source/blender/compositor/operations/COM_CryptomatteOperation.cpp
M	source/blender/compositor/operations/COM_CurveBaseOperation.cpp
M	source/blender/compositor/operations/COM_DenoiseOperation.cpp
M	source/blender/compositor/operations/COM_DespeckleOperation.cpp
M	source/blender/compositor/operations/COM_DifferenceMatteOperation.cpp
M	source/blender/compositor/operations/COM_DilateErodeOperation.cpp
M	source/blender/compositor/operations/COM_DirectionalBlurOperation.cpp
M	source/blender/compositor/operations/COM_DisplaceOperation.cpp
M	source/blender/compositor/operations/COM_DisplaceSimpleOperation.cpp
M	source/blender/compositor/operations/COM_DistanceRGBMatteOperation.cpp
M	source/blender/compositor/operations/COM_DistanceYCCMatteOperation.cpp
M	source/blender/compositor/operations/COM_DotproductOperation.cpp
M	source/blender/compositor/operations/COM_DoubleEdgeMaskOperation.cpp
M	source/blender/compositor/operations/COM_EllipseMaskOperation.cpp
M	source/blender/compositor/operations/COM_FastGaussianBlurOperation.cpp
M	source/blender/compositor/operations/COM_FlipOperation.cpp
M	source/blender/compositor/operations/COM_GammaCorrectOperation.cpp
M	source/blender/compositor/operations/COM_GammaOperation.cpp
M	source/blender/compositor/operations/COM_GlareBaseOperation.cpp
M	source/blender/compositor/operations/COM_GlareThresholdOperation.cpp
M	source/blender/compositor/operations/COM_HueSaturationValueCorrectOperation.cpp
M	source/blender/compositor/operations/COM_IDMaskOperation.cpp
M	source/blender/compositor/operations/COM_ImageOperation.cpp
M	source/blender/compositor/operations/COM_InpaintOperation.cpp
M	source/blender/compositor/operations/COM_InvertOperation.cpp
M	source/blender/compositor/operations/COM_KeyingBlurOperation.cpp
M	source/blender/compositor/operations/COM_KeyingClipOperation.cpp
M	source/blender/compositor/operations/COM_KeyingDespillOperation.cpp
M	source/blender/compositor/operations/COM_KeyingOperation.cpp
M	source/blender/compositor/operations/COM_KeyingScreenOperation.cpp
M	source/blender/compositor/operations/COM_KeyingSetAlphaOperation.cpp
M	source/blender/compositor/operations/COM_LuminanceMatteOperation.cpp
M	source/blender/compositor/operations/COM_MapRangeOperation.cpp
M	source/blender/compositor/operations/COM_MapUVOperation.cpp
M	source/blender/compositor/operations/COM_MapValueOperation.cpp
M	source/blender/compositor/operations/COM_MaskOperation.cpp
M	source/blender/compositor/operations/COM_MathBaseOperation.cpp
M	source/blender/compositor/operations/COM_MixOperation.cpp
M	source/blender/compositor/operations/COM_MovieClipAttributeOperation.cpp
M	source/blender/compositor/operations/COM_MovieClipOperation.cpp
M	source/blender/compositor/operations/COM_MovieDistortionOperation.cpp
M	source/blender/compositor/operations/COM_MultilayerImageOperation.cpp
M	source/blender/compositor/operations/COM_NormalizeOperation.cpp
M	source/blender/compositor/operations/COM_PixelateOperation.cpp
M	source/blender/compositor/operations/COM_PlaneCornerPinOperation.cpp
M	source/blender/compositor/operations/COM_PlaneDistortCommonOperation.cpp
M	source/blender/compositor/operations/COM_PlaneTrackOperation.h
M	source/blender/compositor/operations/COM_PreviewOperation.cpp
M	source/blender/compositor/operations/COM_ProjectorLensDistortionOperation.cpp
M	source/blender/compositor/operations/COM_ReadBufferOperation.cpp
M	source/blender/compositor/operations/COM_RenderLayersProg.cpp
M	source/blender/compositor/operations/COM_RotateOperation.cpp
M	source/blender/compositor/operations/COM_ScreenLensDistortionOperation.cpp
M	source/blender/compositor/operations/COM_SetAlphaOperation.cpp
M	source/blender/compositor/operations/COM_SetColorOperation.cpp
M	source/blender/compositor/operations/COM_SetSamplerOperation.cpp
M	source/blender/compositor/operations/COM_SetValueOperation.cpp
M	source/blender/compositor/operations/COM_SetVectorOperation.cpp
M	source/blender/compositor/operations/COM_SocketProxyOperation.cpp
M	source/blender/compositor/operations/COM_SplitOperation.cpp
M	source/blender/compositor/operations/COM_SunBeamsOperation.cpp
M	source/blender/compositor/operations/COM_TextureOperation.cpp
M	source/blender/compositor/operations/COM_TonemapOperation.cpp
M	source/blender/compositor/operations/COM_TrackPositionOperation.cpp
M	source/blender/compositor/operations/COM_TranslateOperation.cpp
M	source/blender/compositor/operations/COM_VariableSizeBokehBlurOperation.cpp
M	source/blender/compositor/operations/COM_VectorBlurOperation.cpp
M	source/blender/compositor/operations/COM_VectorCurveOperation.cpp
M	source/blender/compositor/operations/COM_ViewerOperation.cpp
M	source/blender/compositor/operations/COM_WriteBufferOperation.cpp
M	source/blender/compositor/operations/COM_ZCombineOperation.cpp
M	source/blender/depsgraph/intern/builder/deg_builder_relations_drivers.cc
M	source/blender/depsgraph/intern/builder/pipeline.cc
M	source/blender/freestyle/intern/application/AppCanvas.cpp
M	source/blender/freestyle/intern/blender_interface/BlenderStrokeRenderer.cpp
M	source/blender/freestyle/intern/image/ImagePyramid.cpp
M	source/blender/freestyle/intern/scene_graph/IndexedFaceSet.cpp
M	source/blender/freestyle/intern/scene_graph/NodeLight.cpp
M	source/blender/freestyle/intern/stroke/AdvancedStrokeShaders.cpp
M	source/blender/freestyle/intern/stroke/BasicStrokeShaders.cpp
M	source/blender/freestyle/intern/stroke/BasicStrokeShaders.h
M	source/blender/freestyle/intern/stroke/PSStrokeRenderer.cpp
M	source/blender/freestyle/intern/stroke/Stroke.cpp
M	source/blender/freestyle/intern/stroke/TextStrokeRenderer.cpp
M	source/blender/freestyle/intern/view_map/BoxGrid.cpp
M	source/blender/freestyle/intern/view_map/SphericalGrid.cpp
M	source/blender/freestyle/intern/view_map/ViewMapBuilder.cpp
M	source/blender/gpu/opengl/gl_state.cc
M	source/blender/io/collada/collada_internal.cpp
M	source/blender/io/common/intern/abstract_hierarchy_iterator.cc
M	source/blender/io/usd/intern/usd_writer_abstract.cc

===================================================================

diff --git a/.clang-tidy b/.clang-tidy
index 9a1392407b9..f80f484ffdf 100644
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -15,7 +15,6 @@ Checks:  >
 
   -readability-misleading-indentation,
 
-  -readability-redundant-member-init,
   -readability-use-anyofallof,
 
   -readability-function-cognitive-complexity,
diff --git a/source/blender/blenlib/intern/mesh_intersect.cc b/source/blender/blenlib/intern/mesh_intersect.cc
index f6579439919..b1b8bd61b40 100644
--- a/source/blender/blenlib/intern/mesh_intersect.cc
+++ b/source/blender/blenlib/intern/mesh_intersect.cc
@@ -2637,8 +2637,7 @@ struct SubdivideTrisData {
         tm(tm),
         itt_map(itt_map),
         overlap(overlap),
-        arena(arena),
-        overlap_tri_range{}
+        arena(arena)
   {
   }
 };
diff --git a/source/blender/compositor/intern/COM_CPUDevice.cpp b/source/blender/compositor/intern/COM_CPUDevice.cpp
index bcb14e4dbb2..26fe1ba0bc3 100644
--- a/source/blender/compositor/intern/COM_CPUDevice.cpp
+++ b/source/blender/compositor/intern/COM_CPUDevice.cpp
@@ -18,7 +18,7 @@
 
 #include "COM_CPUDevice.h"
 
-CPUDevice::CPUDevice(int thread_id) : Device(), m_thread_id(thread_id)
+CPUDevice::CPUDevice(int thread_id) : m_thread_id(thread_id)
 {
 }
 
diff --git a/source/blender/compositor/intern/COM_SingleThreadedOperation.cpp b/source/blender/compositor/intern/COM_SingleThreadedOperation.cpp
index 029be6d44b1..98239166860 100644
--- a/source/blender/compositor/intern/COM_SingleThreadedOperation.cpp
+++ b/source/blender/compositor/intern/COM_SingleThreadedOperation.cpp
@@ -18,7 +18,7 @@
 
 #include "COM_SingleThreadedOperation.h"
 
-SingleThreadedOperation::SingleThreadedOperation() : NodeOperation()
+SingleThreadedOperation::SingleThreadedOperation()
 {
   this->m_cachedInstance = NULL;
   setComplex(true);
diff --git a/source/blender/compositor/operations/COM_AlphaOverKeyOperation.cpp b/source/blender/compositor/operations/COM_AlphaOverKeyOperation.cpp
index 1be5ef1f2df..668d07c7c3d 100644
--- a/source/blender/compositor/operations/COM_AlphaOverKeyOperation.cpp
+++ b/source/blender/compositor/operations/COM_AlphaOverKeyOperation.cpp
@@ -18,7 +18,7 @@
 
 #include "COM_AlphaOverKeyOperation.h"
 
-AlphaOverKeyOperation::AlphaOverKeyOperation() : MixBaseOperation()
+AlphaOverKeyOperation::AlphaOverKeyOperation()
 {
   /* pass */
 }
diff --git a/source/blender/compositor/operations/COM_AlphaOverMixedOperation.cpp b/source/blender/compositor/operations/COM_AlphaOverMixedOperation.cpp
index 513158dcff6..b8465ab7ccf 100644
--- a/source/blender/compositor/operations/COM_AlphaOverMixedOperation.cpp
+++ b/source/blender/compositor/operations/COM_AlphaOverMixedOperation.cpp
@@ -18,7 +18,7 @@
 
 #include "COM_AlphaOverMixedOperation.h"
 
-AlphaOverMixedOperation::AlphaOverMixedOperation() : MixBaseOperation()
+AlphaOverMixedOperation::AlphaOverMixedOperation()
 {
   this->m_x = 0.0f;
 }
diff --git a/source/blender/compositor/operations/COM_AlphaOverPremultiplyOperation.cpp b/source/blender/compositor/operations/COM_AlphaOverPremultiplyOperation.cpp
index 8e5bce7221f..4510c027d46 100644
--- a/source/blender/compositor/operations/COM_AlphaOverPremultiplyOperation.cpp
+++ b/source/blender/compositor/operations/COM_AlphaOverPremultiplyOperation.cpp
@@ -18,7 +18,7 @@
 
 #include "COM_AlphaOverPremultiplyOperation.h"
 
-AlphaOverPremultiplyOperation::AlphaOverPremultiplyOperation() : MixBaseOperation()
+AlphaOverPremultiplyOperation::AlphaOverPremultiplyOperation()
 {
   /* pass */
 }
diff --git a/source/blender/compositor/operations/COM_AntiAliasOperation.cpp b/source/blender/compositor/operations/COM_AntiAliasOperation.cpp
index 3d5e53feb39..e1e58c9521a 100644
--- a/source/blender/compositor/operations/COM_AntiAliasOperation.cpp
+++ b/source/blender/compositor/operations/COM_AntiAliasOperation.cpp
@@ -112,7 +112,7 @@ static int extrapolate9(float *E0,
 #undef PCPY
 }
 
-AntiAliasOperation::AntiAliasOperation() : NodeOperation()
+AntiAliasOperation::AntiAliasOperation()
 {
   this->addInputSocket(COM_DT_VALUE);
   this->addOutputSocket(COM_DT_VALUE);
diff --git a/source/blender/compositor/operations/COM_BilateralBlurOperation.cpp b/source/blender/compositor/operations/COM_BilateralBlurOperation.cpp
index 8168867a522..93193aef360 100644
--- a/source/blender/compositor/operations/COM_BilateralBlurOperation.cpp
+++ b/source/blender/compositor/operations/COM_BilateralBlurOperation.cpp
@@ -21,7 +21,7 @@
 
 #include "RE_pipeline.h"
 
-BilateralBlurOperation::BilateralBlurOperation() : NodeOperation()
+BilateralBlurOperation::BilateralBlurOperation()
 {
   this->addInputSocket(COM_DT_COLOR);
   this->addInputSocket(COM_DT_COLOR);
diff --git a/source/blender/compositor/operations/COM_BlurBaseOperation.cpp b/source/blender/compositor/operations/COM_BlurBaseOperation.cpp
index ef0f259c592..0d9a1a184d6 100644
--- a/source/blender/compositor/operations/COM_BlurBaseOperation.cpp
+++ b/source/blender/compositor/operations/COM_BlurBaseOperation.cpp
@@ -22,7 +22,7 @@
 
 #include "RE_pipeline.h"
 
-BlurBaseOperation::BlurBaseOperation(DataType data_type) : NodeOperation()
+BlurBaseOperation::BlurBaseOperation(DataType data_type)
 {
   /* data_type is almost always COM_DT_COLOR except for alpha-blur */
   this->addInputSocket(data_type);
diff --git a/source/blender/compositor/operations/COM_BokehBlurOperation.cpp b/source/blender/compositor/operations/COM_BokehBlurOperation.cpp
index c00ef2468c0..987c5946e48 100644
--- a/source/blender/compositor/operations/COM_BokehBlurOperation.cpp
+++ b/source/blender/compositor/operations/COM_BokehBlurOperation.cpp
@@ -22,7 +22,7 @@
 
 #include "RE_pipeline.h"
 
-BokehBlurOperation::BokehBlurOperation() : NodeOperation()
+BokehBlurOperation::BokehBlurOperation()
 {
   this->addInputSocket(COM_DT_COLOR);
   this->addInputSocket(COM_DT_COLOR, COM_SC_NO_RESIZE);
diff --git a/source/blender/compositor/operations/COM_BokehImageOperation.cpp b/source/blender/compositor/operations/COM_BokehImageOperation.cpp
index 3fd9a77d879..be3f637e4bb 100644
--- a/source/blender/compositor/operations/COM_BokehImageOperation.cpp
+++ b/source/blender/compositor/operations/COM_BokehImageOperation.cpp
@@ -19,7 +19,7 @@
 #include "COM_BokehImageOperation.h"
 #include "BLI_math.h"
 
-BokehImageOperation::BokehImageOperation() : NodeOperation()
+BokehImageOperation::BokehImageOperation()
 {
   this->addOutputSocket(COM_DT_COLOR);
   this->m_deleteData = false;
diff --git a/source/blender/compositor/operations/COM_BoxMaskOperation.cpp b/source/blender/compositor/operations/COM_BoxMaskOperation.cpp
index c5f0108bf5a..d7d779ed0bb 100644
--- a/source/blender/compositor/operations/COM_BoxMaskOperation.cpp
+++ b/source/blender/compositor/operations/COM_BoxMaskOperation.cpp
@@ -20,7 +20,7 @@
 #include "BLI_math.h"
 #include "DNA_node_types.h"
 
-BoxMaskOperation::BoxMaskOperation() : NodeOperation()
+BoxMaskOperation::BoxMaskOperation()
 {
   this->addInputSocket(COM_DT_VALUE);
   this->addInputSocket(COM_DT_VALUE);
diff --git a/source/blender/compositor/operations/COM_BrightnessOperation.cpp b/source/blender/compositor/operations/COM_BrightnessOperation.cpp
index b6c22029899..c72d227138d 100644
--- a/source/blender/compositor/operations/COM_BrightnessOperation.cpp
+++ b/source/blender/compositor/operations/COM_BrightnessOperation.cpp
@@ -18,7 +18,7 @@
 
 #include "COM_BrightnessOperation.h"
 
-BrightnessOperation::BrightnessOperation() : NodeOperation()
+BrightnessOperation::BrightnessOperation()
 {
   this->addInputSocket(COM_DT_COLOR);
   this->addInputSocket(COM_DT_VALUE);
diff --git a/source/blender/compositor/operations/COM_CalculateMeanOperation.cpp b/source/blender/compositor/operations/COM_CalculateMeanOperation.cpp
index ecd61e95f43..115d415a8f8 100644
--- a/source/blender/compositor/operations/COM_CalculateMeanOperation.cpp
+++ b/source/blender/compositor/operations/COM_CalculateMeanOperation.cpp
@@ -22,7 +22,7 @@
 
 #include "IMB_colormanagement.h"
 
-CalculateMeanOperation::CalculateMeanOperation() : NodeOperation()
+CalculateMeanOperation::CalculateMeanOperation()
 {
   this->addInputSocket(COM_DT_COLOR, COM_SC_NO_RESIZE);
   this->addOutputSocket(COM_DT_VALUE);
diff --git a/source/blender/compositor/operations/COM_CalculateStandardDeviationOperation.cpp b/source/blender/compositor/operations/COM_CalculateStandardDeviationOperation.cpp
index 059040d6f05..476626ffcb6 100644
--- a/source/blender/compositor/operations/COM_CalculateStandardDeviationOperation.cpp
+++ b/source/blender/compositor/operations/COM_CalculateStandardDeviationOperation.cpp
@@ -23,7 +23,6 @@
 #include "IMB_colormanagement.h"
 
 CalculateStandardDeviationOperation::CalculateStandardDeviationOperation()
-    : CalculateMeanOperation()
 {
   /* pass */
 }
diff --git a/source/blender/compositor/operations/COM_ChangeHSVOperation.cpp b/source/blender/compositor/operations/COM_ChangeHSVOperation.cpp
index ddbc400777f..b7ee3ad6de3 100644
--- a/source/blender/compositor/operations/COM_ChangeHSVOperation.cpp
+++ b/source/blender/compositor/operations/COM_ChangeHSVOperation.cpp
@@ -18,7 +18,7 @@
 
 #include "COM_ChangeHSVOperation.h"
 
-ChangeHSVOperation::ChangeHSVOperation() : NodeOperation()
+ChangeHSVOperation::ChangeHSVOperation()
 {
   this->addInputSocket(COM_DT_COLOR);
   this->addInputSocket(COM_DT_VALUE);
diff --git a/source/blender/compositor/operations/COM_ChannelMatteOperation.cpp b/source/blender/compositor/operations/COM_ChannelMatteOperation.cpp
index cbd46ea71f8..8dcee8c6070 100644
--- a/source/blender/compositor/operations/COM_ChannelMatteOperation.cpp
+++ b/source/blender/compositor/operations/COM_ChannelMatteOperation.cpp
@@ -19,7 +19,7 @@
 #include "COM_ChannelMatteOperation.h"
 #include "BLI_math.h"
 
-ChannelMatteOperation::ChannelMatteOperation() : NodeOperation()
+ChannelMatteOperation::ChannelMatteOperation()
 {
   addInputSocket(COM_DT_COLOR);
   addOutputSocket(COM_DT_VALUE);
diff --git a/source/blender/compositor/operations/COM_ChromaMatteOperation.cpp b/source/blender/compositor/operations/COM_ChromaMatteOperation.cpp
index 62dc74d2092..c55f434801d 100644
--- a/source/blender/compositor/operations/COM_ChromaMatteOperation.cpp
+++ b/source/blender/compositor/operations/COM_ChromaMatteOperation.cpp
@@ -19,7 +19,

@@ Diff output truncated at 10240 characters. @@



More information about the Bf-blender-cvs mailing list