[rustc.git] / src / tools / clippy / src / docs / redundant_closure.txt

### What it does
Checks for closures which just call another function where
the function can be called directly. `unsafe` functions or calls where types
get adjusted are ignored.

### Why is this bad?
Needlessly creating a closure adds code for no benefit
and gives the optimizer more work.

### Known problems
If creating the closure inside the closure has a side-
effect then moving the closure creation out will change when that side-
effect runs.
See [#1439](https://github.com/rust-lang/rust-clippy/issues/1439) for more details.

### Example
```
xs.map(|x| foo(x))
```

Use instead:
```
// where `foo(_)` is a plain function that takes the exact argument type of `x`.
xs.map(foo)
```
Commit	Line	Data
f2b60f7d FG	1	### What it does
	2	Checks for closures which just call another function where
	3	the function can be called directly. `unsafe` functions or calls where types
	4	get adjusted are ignored.
	5
	6	### Why is this bad?
	7	Needlessly creating a closure adds code for no benefit
	8	and gives the optimizer more work.
	9
	10	### Known problems
	11	If creating the closure inside the closure has a side-
	12	effect then moving the closure creation out will change when that side-
	13	effect runs.
	14	See [#1439](https://github.com/rust-lang/rust-clippy/issues/1439) for more details.
	15
	16	### Example
	17	```
	18	xs.map(\|x\| foo(x))
	19	```
	20
	21	Use instead:
	22	```
	23	// where `foo(_)` is a plain function that takes the exact argument type of `x`.
	24	xs.map(foo)
	25	```