error: consider implementing `TryFrom` instead
- --> $DIR/fallible_impl_from.rs:7:1
+ --> $DIR/fallible_impl_from.rs:5:1
|
-7 | / impl From<String> for Foo {
-8 | | fn from(s: String) -> Self {
-9 | | Foo(s.parse().unwrap())
-10 | | }
-11 | | }
+LL | / impl From<String> for Foo {
+LL | | fn from(s: String) -> Self {
+LL | | Foo(s.parse().unwrap())
+LL | | }
+LL | | }
| |_^
|
-note: lint level defined here
- --> $DIR/fallible_impl_from.rs:3:9
+note: the lint level is defined here
+ --> $DIR/fallible_impl_from.rs:1:9
|
-3 | #![deny(clippy::fallible_impl_from)]
+LL | #![deny(clippy::fallible_impl_from)]
| ^^^^^^^^^^^^^^^^^^^^^^^^^^
= help: `From` is intended for infallible conversions only. Use `TryFrom` if there's a possibility for the conversion to fail.
note: potential failure(s)
- --> $DIR/fallible_impl_from.rs:9:13
+ --> $DIR/fallible_impl_from.rs:7:13
|
-9 | Foo(s.parse().unwrap())
+LL | Foo(s.parse().unwrap())
| ^^^^^^^^^^^^^^^^^^
error: consider implementing `TryFrom` instead
- --> $DIR/fallible_impl_from.rs:30:1
+ --> $DIR/fallible_impl_from.rs:26:1
|
-30 | / impl From<usize> for Invalid {
-31 | | fn from(i: usize) -> Invalid {
-32 | | if i != 42 {
-33 | | panic!();
+LL | / impl From<usize> for Invalid {
+LL | | fn from(i: usize) -> Invalid {
+LL | | if i != 42 {
+LL | | panic!();
... |
-36 | | }
-37 | | }
+LL | | }
+LL | | }
| |_^
|
= help: `From` is intended for infallible conversions only. Use `TryFrom` if there's a possibility for the conversion to fail.
note: potential failure(s)
- --> $DIR/fallible_impl_from.rs:33:13
+ --> $DIR/fallible_impl_from.rs:29:13
|
-33 | panic!();
+LL | panic!();
| ^^^^^^^^^
- = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)
+ = note: this error originates in a macro (in Nightly builds, run with -Z macro-backtrace for more info)
error: consider implementing `TryFrom` instead
- --> $DIR/fallible_impl_from.rs:39:1
+ --> $DIR/fallible_impl_from.rs:35:1
|
-39 | / impl From<Option<String>> for Invalid {
-40 | | fn from(s: Option<String>) -> Invalid {
-41 | | let s = s.unwrap();
-42 | | if !s.is_empty() {
+LL | / impl From<Option<String>> for Invalid {
+LL | | fn from(s: Option<String>) -> Invalid {
+LL | | let s = s.unwrap();
+LL | | if !s.is_empty() {
... |
-48 | | }
-49 | | }
+LL | | }
+LL | | }
| |_^
|
= help: `From` is intended for infallible conversions only. Use `TryFrom` if there's a possibility for the conversion to fail.
note: potential failure(s)
- --> $DIR/fallible_impl_from.rs:41:17
+ --> $DIR/fallible_impl_from.rs:37:17
|
-41 | let s = s.unwrap();
+LL | let s = s.unwrap();
| ^^^^^^^^^^
-42 | if !s.is_empty() {
-43 | panic!(42);
+LL | if !s.is_empty() {
+LL | panic!(42);
| ^^^^^^^^^^^
-44 | } else if s.parse::<u32>().unwrap() != 42 {
+LL | } else if s.parse::<u32>().unwrap() != 42 {
| ^^^^^^^^^^^^^^^^^^^^^^^^^
-45 | panic!("{:?}", s);
+LL | panic!("{:?}", s);
| ^^^^^^^^^^^^^^^^^^
- = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)
+ = note: this error originates in a macro (in Nightly builds, run with -Z macro-backtrace for more info)
error: consider implementing `TryFrom` instead
- --> $DIR/fallible_impl_from.rs:57:1
+ --> $DIR/fallible_impl_from.rs:53:1
|
-57 | / impl<'a> From<&'a mut <Box<u32> as ProjStrTrait>::ProjString> for Invalid {
-58 | | fn from(s: &'a mut <Box<u32> as ProjStrTrait>::ProjString) -> Invalid {
-59 | | if s.parse::<u32>().ok().unwrap() != 42 {
-60 | | panic!("{:?}", s);
+LL | / impl<'a> From<&'a mut <Box<u32> as ProjStrTrait>::ProjString> for Invalid {
+LL | | fn from(s: &'a mut <Box<u32> as ProjStrTrait>::ProjString) -> Invalid {
+LL | | if s.parse::<u32>().ok().unwrap() != 42 {
+LL | | panic!("{:?}", s);
... |
-63 | | }
-64 | | }
+LL | | }
+LL | | }
| |_^
|
= help: `From` is intended for infallible conversions only. Use `TryFrom` if there's a possibility for the conversion to fail.
note: potential failure(s)
- --> $DIR/fallible_impl_from.rs:59:12
+ --> $DIR/fallible_impl_from.rs:55:12
|
-59 | if s.parse::<u32>().ok().unwrap() != 42 {
+LL | if s.parse::<u32>().ok().unwrap() != 42 {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-60 | panic!("{:?}", s);
+LL | panic!("{:?}", s);
| ^^^^^^^^^^^^^^^^^^
- = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)
+ = note: this error originates in a macro (in Nightly builds, run with -Z macro-backtrace for more info)
error: aborting due to 4 previous errors