Ything-RepoMirrors/poem
1
0
Fork 0
mirror of https://github.com/poem-web/poem.git synced 2026-01-25 04:18:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove unnecessary ok() from openapi-derive (#590)

Browse Source 
Clippy complains about this in crates using the derive(Union)
This commit is contained in:
Adrian Müller (DTT)
2023-06-21 13:17:22 +02:00
committed by GitHub
parent 8b55dd1185
commit bb33f5555e
1 changed files with 4 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
poem-openapi-derive/src/union.rs
View File

@@ -105,18 +105,16 @@ pub(crate) fn generate(args: DeriveInput) -> GeneratorResult<TokenStream> {
} else if !args.one_of {
// any of
from_json.push(quote! {
if let ::std::option::Option::Some(obj) = <#object_ty as #crate_name::types::ParseFromJSON>::parse_from_json(::std::option::Option::Some(::std::clone::Clone::clone(&value)))
.map(Self::#item_ident)
.ok() {
if let ::std::result::Result::Ok(obj) = <#object_ty as #crate_name::types::ParseFromJSON>::parse_from_json(::std::option::Option::Some(::std::clone::Clone::clone(&value)))
.map(Self::#item_ident) {
return ::std::result::Result::Ok(obj);
}
});
} else {
// one of
from_json.push(quote! {
if let ::std::option::Option::Some(obj) = <#object_ty as #crate_name::types::ParseFromJSON>::parse_from_json(::std::option::Option::Some(::std::clone::Clone::clone(&value)))
.map(Self::#item_ident)
.ok() {
if let ::std::result::Result::Ok(obj) = <#object_ty as #crate_name::types::ParseFromJSON>::parse_from_json(::std::option::Option::Some(::std::clone::Clone::clone(&value)))
.map(Self::#item_ident) {
if res_obj.is_some() {
return ::std::result::Result::Err(#crate_name::types::ParseError::expected_type(value));
}