Use map instead of ?

novacrazy · novacrazy · commit 21fdfe8478bd · 2024-01-16T13:49:32.000-06:00
diff --git a/postgres-types/src/smol_str_02.rs b/postgres-types/src/smol_str_02.rs
@@ -6,7 +6,7 @@ use crate::{FromSql, IsNull, ToSql, Type};
 
 impl<'a> FromSql<'a> for SmolStr {
     fn from_sql(ty: &Type, raw: &'a [u8]) -> Result<Self, Box<dyn Error + Sync + Send>> {
-        Ok(SmolStr::new(<&str as FromSql>::from_sql(ty, raw)?))
+        <&str as FromSql>::from_sql(ty, raw).map(SmolStr::new)
     }
 
     fn accepts(ty: &Type) -> bool {

Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@ use crate::{FromSql, IsNull, ToSql, Type};`
`6`	`6`
`7`	`7`	`impl<'a> FromSql<'a> for SmolStr {`
`8`	`8`	`fn from_sql(ty: &Type, raw: &'a [u8]) -> Result<Self, Box<dyn Error + Sync + Send>> {`
`9`		`- Ok(SmolStr::new(<&str as FromSql>::from_sql(ty, raw)?))`
	`9`	`+ <&str as FromSql>::from_sql(ty, raw).map(SmolStr::new)`
`10`	`10`	`}`
`11`	`11`
`12`	`12`	`fn accepts(ty: &Type) -> bool {`