mirror of
https://github.com/rust-lang/rust-clippy
synced 2024-11-27 15:11:30 +00:00
finishing touches, move fixed ICEs to ui tests
This commit is contained in:
parent
fb95df70a7
commit
74bc964e60
2 changed files with 16 additions and 7 deletions
|
@ -104,15 +104,18 @@ fn main() {}
|
||||||
|
|
||||||
struct D;
|
struct D;
|
||||||
|
|
||||||
|
/* FIXME(effects)
|
||||||
impl const Drop for D {
|
impl const Drop for D {
|
||||||
fn drop(&mut self) {
|
fn drop(&mut self) {
|
||||||
todo!();
|
todo!();
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
*/
|
||||||
|
|
||||||
// Lint this, since it can be dropped in const contexts
|
// Lint this, since it can be dropped in const contexts
|
||||||
// FIXME(effects)
|
// FIXME(effects)
|
||||||
fn d(this: D) {}
|
fn d(this: D) {}
|
||||||
|
//~^ ERROR: this could be a `const fn`
|
||||||
|
|
||||||
mod msrv {
|
mod msrv {
|
||||||
struct Foo(*const u8, &'static u8);
|
struct Foo(*const u8, &'static u8);
|
||||||
|
|
|
@ -157,7 +157,13 @@ LL | const fn msrv_1_46() -> i32 {
|
||||||
| +++++
|
| +++++
|
||||||
|
|
||||||
error: this could be a `const fn`
|
error: this could be a `const fn`
|
||||||
--> tests/ui/missing_const_for_fn/could_be_const.rs:122:9
|
--> tests/ui/missing_const_for_fn/could_be_const.rs:117:1
|
||||||
|
|
|
||||||
|
LL | fn d(this: D) {}
|
||||||
|
| ^^^^^^^^^^^^^^^^
|
||||||
|
|
||||||
|
error: this could be a `const fn`
|
||||||
|
--> tests/ui/missing_const_for_fn/could_be_const.rs:125:9
|
||||||
|
|
|
|
||||||
LL | / fn deref_ptr_can_be_const(self) -> usize {
|
LL | / fn deref_ptr_can_be_const(self) -> usize {
|
||||||
LL | |
|
LL | |
|
||||||
|
@ -171,7 +177,7 @@ LL | const fn deref_ptr_can_be_const(self) -> usize {
|
||||||
| +++++
|
| +++++
|
||||||
|
|
||||||
error: this could be a `const fn`
|
error: this could be a `const fn`
|
||||||
--> tests/ui/missing_const_for_fn/could_be_const.rs:127:9
|
--> tests/ui/missing_const_for_fn/could_be_const.rs:130:9
|
||||||
|
|
|
|
||||||
LL | / fn deref_copied_val(self) -> usize {
|
LL | / fn deref_copied_val(self) -> usize {
|
||||||
LL | |
|
LL | |
|
||||||
|
@ -185,7 +191,7 @@ LL | const fn deref_copied_val(self) -> usize {
|
||||||
| +++++
|
| +++++
|
||||||
|
|
||||||
error: this could be a `const fn`
|
error: this could be a `const fn`
|
||||||
--> tests/ui/missing_const_for_fn/could_be_const.rs:138:5
|
--> tests/ui/missing_const_for_fn/could_be_const.rs:141:5
|
||||||
|
|
|
|
||||||
LL | / fn union_access_can_be_const() {
|
LL | / fn union_access_can_be_const() {
|
||||||
LL | |
|
LL | |
|
||||||
|
@ -200,7 +206,7 @@ LL | const fn union_access_can_be_const() {
|
||||||
| +++++
|
| +++++
|
||||||
|
|
||||||
error: this could be a `const fn`
|
error: this could be a `const fn`
|
||||||
--> tests/ui/missing_const_for_fn/could_be_const.rs:152:9
|
--> tests/ui/missing_const_for_fn/could_be_const.rs:155:9
|
||||||
|
|
|
|
||||||
LL | / pub fn new(strings: Vec<String>) -> Self {
|
LL | / pub fn new(strings: Vec<String>) -> Self {
|
||||||
LL | | Self { strings }
|
LL | | Self { strings }
|
||||||
|
@ -213,7 +219,7 @@ LL | pub const fn new(strings: Vec<String>) -> Self {
|
||||||
| +++++
|
| +++++
|
||||||
|
|
||||||
error: this could be a `const fn`
|
error: this could be a `const fn`
|
||||||
--> tests/ui/missing_const_for_fn/could_be_const.rs:157:9
|
--> tests/ui/missing_const_for_fn/could_be_const.rs:160:9
|
||||||
|
|
|
|
||||||
LL | / pub fn empty() -> Self {
|
LL | / pub fn empty() -> Self {
|
||||||
LL | | Self { strings: Vec::new() }
|
LL | | Self { strings: Vec::new() }
|
||||||
|
@ -226,7 +232,7 @@ LL | pub const fn empty() -> Self {
|
||||||
| +++++
|
| +++++
|
||||||
|
|
||||||
error: this could be a `const fn`
|
error: this could be a `const fn`
|
||||||
--> tests/ui/missing_const_for_fn/could_be_const.rs:168:9
|
--> tests/ui/missing_const_for_fn/could_be_const.rs:171:9
|
||||||
|
|
|
|
||||||
LL | / pub fn new(text: String) -> Self {
|
LL | / pub fn new(text: String) -> Self {
|
||||||
LL | | let vec = Vec::new();
|
LL | | let vec = Vec::new();
|
||||||
|
@ -239,5 +245,5 @@ help: make the function `const`
|
||||||
LL | pub const fn new(text: String) -> Self {
|
LL | pub const fn new(text: String) -> Self {
|
||||||
| +++++
|
| +++++
|
||||||
|
|
||||||
error: aborting due to 17 previous errors
|
error: aborting due to 18 previous errors
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue