From e701c15c79f0630d8ed56a0d365b8e506b9f8527 Mon Sep 17 00:00:00 2001 From: Niels Dekker Date: Wed, 21 Feb 2024 15:32:28 +0100 Subject: [PATCH] PERF: FUTURE: Default default-constructor of `SymmetricSecondRankTensor` Follow-up to pull request https://github.com/InsightSoftwareConsortium/ITK/pull/4469 commit 755cd10e5421d41f4004be101c888d5ba24be8e4 "PERF: FUTURE: Default default-constructors of `RGBPixel` and `RGBAPixel`" --- .../Core/Common/include/itkSymmetricSecondRankTensor.h | 4 ++++ Modules/Core/Common/test/itkCommonTypeTraitsGTest.cxx | 9 ++++++++- 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/Modules/Core/Common/include/itkSymmetricSecondRankTensor.h b/Modules/Core/Common/include/itkSymmetricSecondRankTensor.h index c193586f3f7..9a8de6111c6 100644 --- a/Modules/Core/Common/include/itkSymmetricSecondRankTensor.h +++ b/Modules/Core/Common/include/itkSymmetricSecondRankTensor.h @@ -104,7 +104,11 @@ class ITK_TEMPLATE_EXPORT SymmetricSecondRankTensor : public FixedArrayFill(0); } +#endif SymmetricSecondRankTensor(const ComponentType & r) { this->Fill(r); } diff --git a/Modules/Core/Common/test/itkCommonTypeTraitsGTest.cxx b/Modules/Core/Common/test/itkCommonTypeTraitsGTest.cxx index 51d8cf3aeac..8a4370d42d7 100644 --- a/Modules/Core/Common/test/itkCommonTypeTraitsGTest.cxx +++ b/Modules/Core/Common/test/itkCommonTypeTraitsGTest.cxx @@ -74,6 +74,13 @@ TEST(CommonTypeTraits, RGBPixelIsPOD) EXPECT_TRUE(std::is_trivial_v); EXPECT_TRUE(std::is_standard_layout_v); } + +TEST(CommonTypeTraits, SymmetricSecondRankTensorIsPOD) +{ + using T = itk::SymmetricSecondRankTensor; + EXPECT_TRUE(std::is_trivial_v); + EXPECT_TRUE(std::is_standard_layout_v); +} #else TEST(CommonTypeTraits, RGBAPixelIsNotPOD) { @@ -90,7 +97,6 @@ TEST(CommonTypeTraits, RGBPixelIsNotPOD) EXPECT_FALSE(std::is_trivial_v); EXPECT_TRUE(std::is_standard_layout_v); } -#endif TEST(CommonTypeTraits, SymmetricSecondRankTensorIsNotPOD) { @@ -99,6 +105,7 @@ TEST(CommonTypeTraits, SymmetricSecondRankTensorIsNotPOD) EXPECT_FALSE(std::is_trivial_v); EXPECT_TRUE(std::is_standard_layout_v); } +#endif /************ Second Generation FixedArray *************/ /* Derived from Point */