Auto merge of #9709 - koka831:chore/remove-unnecessary-files, r=flip1995
chore: remove unnecessary files removes document text files that are no longer needed by #9541. changelog: none r? `@Alexendoo`
This commit is contained in:
commit
df67ebba5e
@ -1,20 +0,0 @@
|
|||||||
### What it does
|
|
||||||
|
|
||||||
Warn of cases where `let...else` could be used
|
|
||||||
|
|
||||||
### Why is this bad?
|
|
||||||
|
|
||||||
`let...else` provides a standard construct for this pattern
|
|
||||||
that people can easily recognize. It's also more compact.
|
|
||||||
|
|
||||||
### Example
|
|
||||||
|
|
||||||
```
|
|
||||||
let v = if let Some(v) = w { v } else { return };
|
|
||||||
```
|
|
||||||
|
|
||||||
Could be written:
|
|
||||||
|
|
||||||
```
|
|
||||||
let Some(v) = w else { return };
|
|
||||||
```
|
|
@ -1,22 +0,0 @@
|
|||||||
### What it does
|
|
||||||
|
|
||||||
Checks for jumps to the start of a stream that implements `Seek`
|
|
||||||
and uses the `seek` method providing `Start` as parameter.
|
|
||||||
|
|
||||||
### Why is this bad?
|
|
||||||
|
|
||||||
Readability. There is a specific method that was implemented for
|
|
||||||
this exact scenario.
|
|
||||||
|
|
||||||
### Example
|
|
||||||
```
|
|
||||||
fn foo<T: io::Seek>(t: &mut T) {
|
|
||||||
t.seek(io::SeekFrom::Start(0));
|
|
||||||
}
|
|
||||||
```
|
|
||||||
Use instead:
|
|
||||||
```
|
|
||||||
fn foo<T: io::Seek>(t: &mut T) {
|
|
||||||
t.rewind();
|
|
||||||
}
|
|
||||||
```
|
|
Loading…
Reference in New Issue
Block a user