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

Don't clone field names when recursing down in selection sets

Browse source
This commit is contained in:
Magnus Hallin 2016-12-28 11:51:37 +01:00
parent 13903d7628
commit d7e98d955a
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/executor.rs
View file

@ -27,7 +27,7 @@ pub struct Registry {
#[derive(Clone)] #[derive(Clone)]
pub enum FieldPath<'a> { pub enum FieldPath<'a> {
Root(SourcePosition), Root(SourcePosition),
Field(String, SourcePosition, &'a FieldPath<'a>), Field(&'a str, SourcePosition, &'a FieldPath<'a>),
} }
/// Query execution engine /// Query execution engine
@ -187,7 +187,7 @@ impl<'a, CtxT> Executor<'a, CtxT> {
#[doc(hidden)] #[doc(hidden)]
pub fn sub_executor( pub fn sub_executor(
&self, &self,
field_name: Option<String>, field_name: Option<&'a str>,
location: SourcePosition, location: SourcePosition,
selection_set: Option<&'a [Selection]>, selection_set: Option<&'a [Selection]>,
) )
@ -251,7 +251,7 @@ impl<'a> FieldPath<'a> {
FieldPath::Root(_) => (), FieldPath::Root(_) => (),
FieldPath::Field(ref name, _, ref parent) => { FieldPath::Field(ref name, _, ref parent) => {
parent.construct_path(acc); parent.construct_path(acc);
acc.push(name.clone()); acc.push((*name).to_owned());
} }
} }
} }

2
src/types/base.rs
View file

@ -325,7 +325,7 @@ fn resolve_selection_set_into<T, CtxT>(
let exec_vars = executor.variables(); let exec_vars = executor.variables();
let mut sub_exec = executor.sub_executor( let mut sub_exec = executor.sub_executor(
Some(response_name.clone()), Some(response_name),
start_pos.clone(), start_pos.clone(),
f.selection_set.as_ref().map(|v| &v[..])); f.selection_set.as_ref().map(|v| &v[..]));