remove warnings - #2885
remove warnings#2885gouarin wants to merge 3 commits into
Conversation
Sorry, something went wrong.
| EXPECT_TRUE(allclose(cmplexpected_angle, cmplres_angle)); | ||
|
|
||
| using assign_t_angle = xassign_traits<xarray<double>, decltype(cmplres_angle)>; | ||
| (void) sizeof(assign_t_angle); // to avoid unused warning |
There was a problem hiding this comment.
Please move the definition where it is used instead of this horrible hack.
Sorry, something went wrong.
| EXPECT_TRUE(allclose(cmplexpected_conj, cmplres_conj)); | ||
|
|
||
| using assign_t_conj = xassign_traits<xarray<std::complex<double>>, decltype(cmplres_conj)>; | ||
| (void) sizeof(assign_t_conj); // to avoid unused warning |
There was a problem hiding this comment.
Same remark here
Sorry, something went wrong.
| }; | ||
|
|
||
| using assign_t_norm = xassign_traits<xarray<double>, decltype(cmplres_norm)>; | ||
| (void) sizeof(assign_t_norm); // to avoid unused warning |
There was a problem hiding this comment.
Same here.
Sorry, something went wrong.
| } | ||
|
|
||
| using assign_t_arg = xassign_traits<xarray<double>, decltype(cmplres)>; | ||
| (void) sizeof(assign_t_arg); // to avoid unused warning |
There was a problem hiding this comment.
Same here.
Sorry, something went wrong.
|
|
||
| #define TEST_VALUE_HAS_VALUE(INPUT, V_TYPE, OPTIONAL) \ | ||
| using result_type = std::conditional_t<OPTIONAL, xtl::xoptional<double>, double>; \ | ||
| (void) sizeof(result_type); \ |
There was a problem hiding this comment.
This is not required since result_type is used just below.
Sorry, something went wrong.
| EXPECT_EQ(a, par); | ||
| } | ||
| using assign_traits = xassign_traits<xarray<double>, decltype(av)>; | ||
| (void) sizeof(assign_traits); // to avoid unused warning |
There was a problem hiding this comment.
Please move the definition where it is used instead.
Sorry, something went wrong.
| // Suppress unused typedef warnings | ||
| (void) sizeof(ctes); | ||
| (void) sizeof(carr); | ||
| (void) sizeof(cfix); |
There was a problem hiding this comment.
These 3 lines can be removed.
Sorry, something went wrong.
| auto vxa = view(xt::arange<double>(100), range(0, 10)); | ||
|
|
||
| using assign_traits = xassign_traits<decltype(vxt), decltype(b)>; | ||
| (void) sizeof(assign_traits); // suppress unused typedef warning |
There was a problem hiding this comment.
Please move the definition where it is used.
Sorry, something went wrong.
Checklist
Description