diff options
| author | Nicholas-Baron <nicholas.baron.ten@gmail.com> | 2020-11-06 13:24:55 -0800 |
|---|---|---|
| committer | Nicholas-Baron <nicholas.baron.ten@gmail.com> | 2020-11-10 20:07:47 -0800 |
| commit | a5ec85758b4b425a08b3401422543fe756f980b2 (patch) | |
| tree | c4d529f8cba1949823252382141d3e806d2e1bab /compiler/rustc_llvm/llvm-wrapper/PassWrapper.cpp | |
| parent | 65ae469f8d422fd758c37ae486a6384fe2a17030 (diff) | |
| download | rust-a5ec85758b4b425a08b3401422543fe756f980b2.tar.gz rust-a5ec85758b4b425a08b3401422543fe756f980b2.zip | |
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.
Diffstat (limited to 'compiler/rustc_llvm/llvm-wrapper/PassWrapper.cpp')
0 files changed, 0 insertions, 0 deletions