yume/juniper
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Refactor code to remove async move closure which may result in double boxing.

Browse source
This commit is contained in:
jawang94 2024-05-03 22:17:12 -07:00
parent 2682ee1418
commit 213cb8de1a
No known key found for this signature in database
1 changed files with 19 additions and 21 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

40
juniper/src/types/async_await.rs
View file

@ -258,31 +258,29 @@ where
let is_non_null = meta_field.field_type.is_non_null();
let response_name = response_name.to_string();
async_values.push_back(AsyncValueFuture::Field(async move {
// TODO: implement custom future type instead of
// two-level boxing.
let res = instance
.resolve_field_async(info, f.name.item, &args, &sub_exec)
.await;
let res = instance
.resolve_field_async(info, f.name.item, &args, &sub_exec)
.await;
let value = match res {
Ok(Value::Null) if is_non_null => None,
Ok(v) => Some(v),
Err(e) => {
sub_exec.push_error_at(e, pos);
if is_non_null {
None
} else {
Some(Value::null())
}
let value = match res {
Ok(Value::Null) if is_non_null => None,
Ok(v) => Some(v),
Err(e) => {
sub_exec.push_error_at(e, pos);
if is_non_null {
None
} else {
Some(Value::null())
}
};
AsyncValue::Field(AsyncField {
}
};
async_values.push_back(AsyncValueFuture::Field(future::ready(AsyncValue::Field(
AsyncField {
name: response_name,
value,
})
}));
},
))));
}
Selection::FragmentSpread(Spanning {