[wip] clang-tidy fixes

AIDASoft · Feb 11, 2022 · dffe4f1 · dffe4f1
1 parent b1d08a0
commit dffe4f1
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/include/podio/Association.h b/include/podio/Association.h
@@ -77,7 +77,7 @@ class AssociationT {
   /// Destructor
   ~AssociationT() {
     if (m_obj) {
-      m_obj->release();
+      m_obj->release(); // NOLINT(clang-analyzer-cplusplus.NewDelete) issue #174
     }
   }
 

diff --git a/include/podio/AssociationSIOBlock.h b/include/podio/AssociationSIOBlock.h
@@ -75,4 +75,4 @@ class AssociationSIOBlock : public podio::SIOBlock {
 
 } // namespace podio
 
-#endif // PODIO_ASSOCIATIONBLOCK_H
+#endif // PODIO_ASSOCIATIONSIOBLOCK_H
diff --git a/tests/associations.cpp b/tests/associations.cpp
@@ -27,7 +27,8 @@ TEST_CASE("Association basics", "[associations]") {
   auto cluster = MutableExampleCluster();
   auto hit = MutableExampleHit();
 
-  auto mutAssoc = TestMutA();
+  auto mutAssoc = TestMutA(); // NOLINT(clang-analyzer-cplusplus.NewDelete) necessary due to clang-tidy and Catch2
+                              // SECTIONs interaction (common setup seems to confuse clang-tidy)
   mutAssoc.setWeight(3.14f);
   mutAssoc.setFrom(hit);
   mutAssoc.setTo(cluster);
Original file line number	Diff line number	Diff line change
Expand Up		@@ -75,4 +75,4 @@ class AssociationSIOBlock : public podio::SIOBlock {

		} // namespace podio

		#endif // PODIO_ASSOCIATIONBLOCK_H
		#endif // PODIO_ASSOCIATIONSIOBLOCK_H