rust/rustc_hir at d486bfcbff107e8a6769e00c59d02b13c664b6ee - rust

History

Nicholas-Baron 261ca04c92 Changed unwrap_or to unwrap_or_else in some places. The discussion seems to have resolved that this lint is a bit "noisy" in that applying it in all places would result in a reduction in readability. A few of the trivial functions (like `Path::new`) are fine to leave outside of closures. The general rule seems to be that anything that is obviously an allocation (`Box`, `Vec`, `vec![]`) should be in a closure, even if it is a 0-sized allocation.	2020-11-10 20:07:47 -08:00
..
src	Changed unwrap_or to unwrap_or_else in some places.	2020-11-10 20:07:47 -08:00
Cargo.toml	hir: replace `lazy_static` by `SyncLazy` from std	2020-09-01 22:06:47 +01:00

Nicholas-Baron 261ca04c92 Changed unwrap_or to unwrap_or_else in some places.

The discussion seems to have resolved that this lint is a bit "noisy" in
that applying it in all places would result in a reduction in
readability.

A few of the trivial functions (like `Path::new`) are fine to leave
outside of closures.

The general rule seems to be that anything that is obviously an
allocation (`Box`, `Vec`, `vec![]`) should be in a closure, even if it
is a 0-sized allocation.

2020-11-10 20:07:47 -08:00

src Changed unwrap_or to unwrap_or_else in some places. 2020-11-10 20:07:47 -08:00

Cargo.toml hir: replace lazy_static by SyncLazy from std 2020-09-01 22:06:47 +01:00