From d7c99d55daeaa76f482444e95beb99f5744ced4e Mon Sep 17 00:00:00 2001 From: Philippe-Cholet <44676486+Philippe-Cholet@users.noreply.github.com> Date: Thu, 16 May 2024 12:29:44 +0200 Subject: [PATCH] `TupleCombinations` is not lazy but must be used nonetheless --- src/adaptors/mod.rs | 2 +- tests/laziness.rs | 9 +++++++-- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/src/adaptors/mod.rs b/src/adaptors/mod.rs index 22f7aa363..52e36c48b 100644 --- a/src/adaptors/mod.rs +++ b/src/adaptors/mod.rs @@ -612,7 +612,7 @@ where /// See [`.tuple_combinations()`](crate::Itertools::tuple_combinations) for more /// information. #[derive(Clone, Debug)] -#[must_use = "iterator adaptors are lazy and do nothing unless consumed"] +#[must_use = "this iterator adaptor is not lazy but does nearly nothing unless consumed"] pub struct TupleCombinations where I: Iterator, diff --git a/tests/laziness.rs b/tests/laziness.rs index 7575e3786..c559d33ad 100644 --- a/tests/laziness.rs +++ b/tests/laziness.rs @@ -196,10 +196,15 @@ must_use_tests! { while_some { let _ = Panicking.map(Some).while_some(); } - #[ignore] - tuple_combinations { + tuple_combinations1 { let _ = Panicking.tuple_combinations::<(_,)>(); + } + #[should_panic] + tuple_combinations2 { let _ = Panicking.tuple_combinations::<(_, _)>(); + } + #[should_panic] + tuple_combinations3 { let _ = Panicking.tuple_combinations::<(_, _, _)>(); } combinations {