Skip to content

Commit

Permalink
perf: Minor performance improvements
Browse files Browse the repository at this point in the history
Signed-off-by: Dmitry Dygalo <[email protected]>
  • Loading branch information
Stranger6667 committed Feb 6, 2025
1 parent 802fcb6 commit 0582581
Showing 1 changed file with 2 additions and 8 deletions.
10 changes: 2 additions & 8 deletions crates/jsonschema-referencing/src/registry.rs
Original file line number Diff line number Diff line change
Expand Up @@ -368,6 +368,7 @@ fn process_meta_schemas(
while let Some((mut base, resource)) = queue.pop_front() {
if let Some(id) = resource.id() {
base = resolution_cache.resolve_against(&base.borrow(), id)?;
resources.insert(base.clone(), resource.clone());
}

// Look for anchors
Expand All @@ -380,9 +381,6 @@ fn process_meta_schemas(
let subresource = InnerResourcePtr::new(contents, resource.draft());
queue.push_back((base.clone(), subresource));
}
if resource.id().is_some() {
resources.insert(base, resource.clone());
}
}
Ok(())
}
Expand Down Expand Up @@ -438,10 +436,9 @@ fn process_resources(

// Process current queue and collect references to external resources
while let Some((mut base, resource)) = queue.pop_front() {
let mut has_id = false;
if let Some(id) = resource.id() {
has_id = true;
base = resolution_cache.resolve_against(&base.borrow(), id)?;
resources.insert(base.clone(), resource.clone());
}

// Look for anchors
Expand All @@ -465,9 +462,6 @@ fn process_resources(
let subresource = InnerResourcePtr::new(contents, resource.draft());
queue.push_back((base.clone(), subresource));
}
if has_id {
resources.insert(base, resource.clone());
}
}
// Retrieve external resources
for uri in external.drain() {
Expand Down

0 comments on commit 0582581

Please sign in to comment.