Skip to content

Commit

Permalink
resolve sonar warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
cvarni committed Oct 2, 2024
1 parent dcdeac4 commit 1c40c90
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
7 changes: 6 additions & 1 deletion Core/include/Acts/Clusterization/Clusterization.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -55,12 +55,14 @@ struct Connect2D {
Connect2D() = default;
explicit Connect2D(bool commonCorner) : conn8{commonCorner} {}
virtual ConnectResult operator()(const Cell& ref, const Cell& iter) const;
virtual ~Connect2D() = default;
};

// Default connection type for 1-D grids: 2-cell connectivity
template <Acts::Ccl::HasRetrievableColumnInfo Cell>
struct Connect1D {
virtual ConnectResult operator()(const Cell& ref, const Cell& iter) const;
virtual ~Connect1D() = default;
};

// Default connection type based on GridDim
Expand All @@ -71,12 +73,15 @@ struct DefaultConnect {
};

template <typename Cell>
struct DefaultConnect<Cell, 1> : public Connect1D<Cell> {};
struct DefaultConnect<Cell, 1> : public Connect1D<Cell> {
virtual ~DefaultConnect() = default;
};

template <typename Cell>
struct DefaultConnect<Cell, 2> : public Connect2D<Cell> {
explicit DefaultConnect(bool commonCorner) : Connect2D<Cell>(commonCorner) {}
DefaultConnect() = default;
virtual ~DefaultConnect() = default;
};

/// @brief labelClusters
Expand Down
1 change: 1 addition & 0 deletions Core/include/Acts/Clusterization/TimedClusterization.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ struct TimedConnect : public Acts::Ccl::DefaultConnect<Cell, N> {
TimedConnect(Acts::ActsScalar time);
TimedConnect(Acts::ActsScalar time, bool conn)
requires(N == 2);
virtual ~TimedConnect() = default;

virtual ConnectResult operator()(const Cell& ref,
const Cell& iter) const override;
Expand Down

0 comments on commit 1c40c90

Please sign in to comment.