Skip to content

Commit

Permalink
fix: Wrong definition of the constraint (#3803)
Browse files Browse the repository at this point in the history
The concept definition was not generic enough to cover for cases with `N != 3ul`
  • Loading branch information
CarloVarni authored Oct 31, 2024
1 parent e7360ed commit 9400320
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions Core/include/Acts/Seeding/SeedFinder.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -38,11 +38,11 @@ concept GridBinCollection =
std::ranges::random_access_range<Coll> &&
std::same_as<typename Coll::value_type, std::size_t>;

template <typename Coll, typename external_t, std::size_t N = 3ul>
concept CollectionStoresSeedsTo = requires(Coll coll, external_t sp) {
Acts::detail::pushBackOrInsertAtEnd(coll,
Acts::Seed<external_t, N>(sp, sp, sp));
};
template <typename collection_t, typename external_t, std::size_t N = 3ul>
concept CollectionStoresSeedsTo =
requires(collection_t coll, Acts::Seed<external_t, N> seed) {
Acts::detail::pushBackOrInsertAtEnd(coll, seed);
};

enum class SpacePointCandidateType : short { eBottom, eTop };

Expand Down

0 comments on commit 9400320

Please sign in to comment.