Skip to content

Commit

Permalink
Fix element type tests for cartesian_product of ints tests to use
Browse files Browse the repository at this point in the history
`distance_t` instead of `long`, as `distance_t` isn't always `long` (for
example on MSVC).
  • Loading branch information
brycelelbach committed Jul 31, 2023
1 parent c3d7880 commit 070d607
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions test/test_cartesian_product.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -204,13 +204,13 @@ constexpr bool test_cartesian_product()
static_assert(flux::bounded_sequence<C const>);
static_assert(flux::sized_sequence<C const>);

static_assert(std::same_as<flux::element_t<C>, std::tuple<long, long, long>>);
static_assert(std::same_as<flux::value_t<C>, std::tuple<long, long, long>>);
static_assert(std::same_as<flux::rvalue_element_t<C>, std::tuple<long, long, long>>);
static_assert(std::same_as<flux::element_t<C>, std::tuple<flux::distance_t, flux::distance_t, flux::distance_t>>);
static_assert(std::same_as<flux::value_t<C>, std::tuple<flux::distance_t, flux::distance_t, flux::distance_t>>);
static_assert(std::same_as<flux::rvalue_element_t<C>, std::tuple<flux::distance_t, flux::distance_t, flux::distance_t>>);

static_assert(std::same_as<flux::element_t<C const>, std::tuple<long, long, long>>);
static_assert(std::same_as<flux::value_t<C const>, std::tuple<long, long, long>>);
static_assert(std::same_as<flux::rvalue_element_t<C const>, std::tuple<long, long, long>>);
static_assert(std::same_as<flux::element_t<C const>, std::tuple<flux::distance_t, flux::distance_t, flux::distance_t>>);
static_assert(std::same_as<flux::value_t<C const>, std::tuple<flux::distance_t, flux::distance_t, flux::distance_t>>);
static_assert(std::same_as<flux::rvalue_element_t<C const>, std::tuple<flux::distance_t, flux::distance_t, flux::distance_t>>);

STATIC_CHECK(flux::size(cart) == 4 * 2 * 3);

Expand Down

0 comments on commit 070d607

Please sign in to comment.