]> git.lizzy.rs Git - rust.git/commitdiff
auto merge of #15561 : huonw/rust/must-use-iterators, r=alexcrichton
authorbors <bors@rust-lang.org>
Thu, 10 Jul 2014 05:16:28 +0000 (05:16 +0000)
committerbors <bors@rust-lang.org>
Thu, 10 Jul 2014 05:16:28 +0000 (05:16 +0000)
Similar to the stability attributes, a type annotated with `#[must_use =
"informative snippet"]` will print the normal warning message along with
"informative snippet". This allows the type author to provide some
guidance about why the type should be used.

---

It can be a little unintuitive that something like `v.iter().map(|x|
println!("{}", x));` does nothing: the majority of the iterator adaptors
are lazy and do not execute anything until something calls `next`, e.g.
a `for` loop, `collect`, `fold`, etc.

The majority of such errors can be seen by someone writing something
like the above, i.e. just calling an iterator adaptor and doing nothing
with it (and doing this is certainly useless), so we can co-opt the
`must_use` lint, using the message functionality to give a hint to the
reason why.

Fixes #14666.

1  2 
src/libcore/iter.rs

Simple merge