rust/tests/ui/drop_bounds.stderr

error: bounds of the form `T: Drop` are useless, use `std::mem::needs_drop` to detect if a type has drop glue
  --> $DIR/drop_bounds.rs:2:11
   |
LL | fn foo<T: Drop>() {}
   |           ^^^^
   |
   = note: `#[deny(clippy::drop_bounds)]` on by default

error: bounds of the form `T: Drop` are useless, use `std::mem::needs_drop` to detect if a type has drop glue
  --> $DIR/drop_bounds.rs:5:8
   |
LL |     T: Drop,
   |        ^^^^

error: aborting due to 2 previous errors
drop_bounds: make lint adhere to lint message convention 2020-07-23 20:05:42 +02:00			error: bounds of the form `T: Drop` are useless, use `std::mem::needs_drop` to detect if a type has drop glue
Add a lint to warn on `T: Drop` bounds What it does: Checks for generics with `std::ops::Drop` as bounds. Why is this bad? `Drop` bounds do not really accomplish anything. A type may have compiler-generated drop glue without implementing the `Drop` trait itself. The `Drop` trait also only has one method, `Drop::drop`, and that function is by fiat not callable in user code. So there is really no use case for using `Drop` in trait bounds. Known problems: None. Example: ```rust fn foo<T: Drop>() {} ``` 2019-02-17 05:28:16 +01:00			`--> $DIR/drop_bounds.rs:2:11`
			`\|`
			`LL \| fn foo<T: Drop>() {}`
			`\| ^^^^`
			`\|`
rustup https://github.com/rust-lang/rust/pull/62764 (was merged as part of https://github.com/rust-lang/rust/pull/62782 ) 2019-07-19 00:35:32 +02:00			= note: `#[deny(clippy::drop_bounds)]` on by default
Add a lint to warn on `T: Drop` bounds What it does: Checks for generics with `std::ops::Drop` as bounds. Why is this bad? `Drop` bounds do not really accomplish anything. A type may have compiler-generated drop glue without implementing the `Drop` trait itself. The `Drop` trait also only has one method, `Drop::drop`, and that function is by fiat not callable in user code. So there is really no use case for using `Drop` in trait bounds. Known problems: None. Example: ```rust fn foo<T: Drop>() {} ``` 2019-02-17 05:28:16 +01:00
drop_bounds: make lint adhere to lint message convention 2020-07-23 20:05:42 +02:00			error: bounds of the form `T: Drop` are useless, use `std::mem::needs_drop` to detect if a type has drop glue
Fix formatting 2019-02-19 03:36:58 +01:00			`--> $DIR/drop_bounds.rs:5:8`
Add a lint to warn on `T: Drop` bounds What it does: Checks for generics with `std::ops::Drop` as bounds. Why is this bad? `Drop` bounds do not really accomplish anything. A type may have compiler-generated drop glue without implementing the `Drop` trait itself. The `Drop` trait also only has one method, `Drop::drop`, and that function is by fiat not callable in user code. So there is really no use case for using `Drop` in trait bounds. Known problems: None. Example: ```rust fn foo<T: Drop>() {} ``` 2019-02-17 05:28:16 +01:00			`\|`
Fix formatting 2019-02-19 03:36:58 +01:00			`LL \| T: Drop,`
			`\| ^^^^`
Add a lint to warn on `T: Drop` bounds What it does: Checks for generics with `std::ops::Drop` as bounds. Why is this bad? `Drop` bounds do not really accomplish anything. A type may have compiler-generated drop glue without implementing the `Drop` trait itself. The `Drop` trait also only has one method, `Drop::drop`, and that function is by fiat not callable in user code. So there is really no use case for using `Drop` in trait bounds. Known problems: None. Example: ```rust fn foo<T: Drop>() {} ``` 2019-02-17 05:28:16 +01:00
			`error: aborting due to 2 previous errors`