about summary refs log tree commit diff
path: root/src/test/codegen
diff options
context:
space:
mode:
authorbors <bors@rust-lang.org>2021-08-15 12:02:24 +0000
committerbors <bors@rust-lang.org>2021-08-15 12:02:24 +0000
commit5449e23e671c500458ba70f5890fc5caa2853845 (patch)
treefd5041faf775210fd0fa8b23436bf997547ae061 /src/test/codegen
parent7c5487dc62d569d4bc32cbe51951668406242137 (diff)
parent3c8eaa8b2c2d594d39810fc2f2a62098c4107c80 (diff)
downloadrust-5449e23e671c500458ba70f5890fc5caa2853845.tar.gz
rust-5449e23e671c500458ba70f5890fc5caa2853845.zip
Auto merge of #7568 - dtolnay-contrib:ifletelse, r=llogiq
Downgrade option_if_let_else to nursery

I believe that this lint's loose understanding of ownership (#5822, #6737) makes it unsuitable to be enabled by default in its current state, even as a pedantic lint.

Additionally the lint has known problems with type inference (#6137), though I may be willing to consider this a non-blocker in isolation if it weren't for the ownership false positives.

A fourth false positive involving const fn: #7567.

But on top of these, for me the biggest issue is I basically fully agree with https://github.com/rust-lang/rust-clippy/issues/6137#issuecomment-705605688. In my experience this lint universally makes code worse even when the resulting code does compile.

---

changelog: remove [`option_if_let_else`] from default set of enabled lints
Diffstat (limited to 'src/test/codegen')
0 files changed, 0 insertions, 0 deletions