소스 검색

fix clang-tidy warning modernize-use-equals-default

Fabian Peter Hammerle 6 년 전
부모
커밋
9e0bb811b9
3개의 변경된 파일5개의 추가작업 그리고 13개의 파일을 삭제
  1. 0 1
      .clang-tidy
  2. 3 6
      lib/HdfItem.cpp
  3. 2 6
      tests/HdfFileTest.cpp

+ 0 - 1
.clang-tidy

@@ -20,7 +20,6 @@ WarningsAsErrors: >-
   -cppcoreguidelines-pro-type-vararg,
   -cppcoreguidelines-special-member-functions,
   -modernize-use-emplace,
-  -modernize-use-equals-default,
   -modernize-use-equals-delete,
   -modernize-use-noexcept,
   -performance-inefficient-vector-operation,

+ 3 - 6
lib/HdfItem.cpp

@@ -32,8 +32,7 @@ std::string hdf4cpp::HdfItem::HdfDatasetItem::getName() const {
 int32 hdf4cpp::HdfItem::HdfDatasetItem::getId() const {
     return id;
 }
-hdf4cpp::HdfItem::HdfDatasetItem::~HdfDatasetItem() {
-}
+hdf4cpp::HdfItem::HdfDatasetItem::~HdfDatasetItem() = default;
 hdf4cpp::HdfItem::HdfGroupItem::HdfGroupItem(int32 id, const HdfDestroyerChain &chain)
     : HdfItemBase(id, VGROUP, chain) {
     char _name[MAX_NAME_LENGTH];
@@ -53,8 +52,7 @@ std::string hdf4cpp::HdfItem::HdfGroupItem::getName() const {
 int32 hdf4cpp::HdfItem::HdfGroupItem::getId() const {
     return id;
 }
-hdf4cpp::HdfItem::HdfGroupItem::~HdfGroupItem() {
-}
+hdf4cpp::HdfItem::HdfGroupItem::~HdfGroupItem() = default;
 hdf4cpp::HdfItem::HdfDataItem::HdfDataItem(int32 id, const HdfDestroyerChain &chain)
     : HdfItemBase(id, VDATA, chain) {
     this->chain.pushBack(new HdfDestroyer(&VSdetach, id));
@@ -62,8 +60,7 @@ hdf4cpp::HdfItem::HdfDataItem::HdfDataItem(int32 id, const HdfDestroyerChain &ch
     VSinquire(id, &nrRecords, &interlace, nullptr, &recordSize, _name);
     name = std::string(_name);
 }
-hdf4cpp::HdfItem::HdfDataItem::~HdfDataItem() {
-}
+hdf4cpp::HdfItem::HdfDataItem::~HdfDataItem() = default;
 hdf4cpp::HdfAttribute hdf4cpp::HdfItem::HdfDataItem::getAttribute(const std::string &name) const {
     return HdfAttribute(new HdfAttribute::HdfDataAttribute(id, name, chain));
 }

+ 2 - 6
tests/HdfFileTest.cpp

@@ -9,11 +9,7 @@ using namespace hdf4cpp;
 
 class HdfFileTest : public ::testing::Test {
   protected:
-    HdfFileTest()
-        : file(std::string(TEST_DATA_PATH) + "small_test.hdf") {
-    }
-
-    HdfFile file;
+    HdfFile file{TEST_DATA_PATH "small_test.hdf"};
 };
 
 TEST_F(HdfFileTest, DatasetValidity) {
@@ -236,4 +232,4 @@ TEST_F(HdfFileTest, VdataDestroyer) {
     std::vector<int32> vec;
     attribute.get(vec);
     ASSERT_EQ(vec, std::vector<int32>({1, 2, 3, 3, 2, 1}));
-}
+}