diff options
Diffstat (limited to '')
-rw-r--r-- | src/tools/clippy/src/docs/borrowed_box.txt | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/src/tools/clippy/src/docs/borrowed_box.txt b/src/tools/clippy/src/docs/borrowed_box.txt new file mode 100644 index 000000000..d7089be66 --- /dev/null +++ b/src/tools/clippy/src/docs/borrowed_box.txt @@ -0,0 +1,19 @@ +### What it does +Checks for use of `&Box<T>` anywhere in the code. +Check the [Box documentation](https://doc.rust-lang.org/std/boxed/index.html) for more information. + +### Why is this bad? +A `&Box<T>` parameter requires the function caller to box `T` first before passing it to a function. +Using `&T` defines a concrete type for the parameter and generalizes the function, this would also +auto-deref to `&T` at the function call site if passed a `&Box<T>`. + +### Example +``` +fn foo(bar: &Box<T>) { ... } +``` + +Better: + +``` +fn foo(bar: &T) { ... } +```
\ No newline at end of file |