fork/Graphite
1
0
Fork 0
mirror of https://github.com/GraphiteEditor/Graphite.git synced 2026-09-29 02:38:13 +08:00
Code Issues Packages Projects Releases Wiki Activity

Reserve the record stack before direct edge evals in executor tests

Browse Source
This commit is contained in:
Dennis Kobert
2026-08-15 12:27:39 +00:00
parent 64fb4eda61
commit 214632c5ec
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
node-graph/interpreted-executor/src/dynamic_executor.rs
+3
View File
@@ -569,6 +569,7 @@ mod test {
let generations = []; let generations = [];
let scope = EvalScope::new(None, None, None, &generations, &arena); let scope = EvalScope::new(None, None, None, &generations, &arena);
let ctx = ContextImpl::root(&scope); let ctx = ContextImpl::root(&scope);
core_types::record::stack::reserve(layout.frame_bytes());
let GPoll::Final(value) = edge.eval(&ctx) else { let GPoll::Final(value) = edge.eval(&ctx) else {
panic!("expected a final record"); panic!("expected a final record");
}; };
@@ -617,6 +618,7 @@ mod test {
.unwrap() .unwrap()
.downcast_record::<graphene_std::list::List<graphene_std::raster_types::Raster<graphene_std::raster_types::CPU>>>() .downcast_record::<graphene_std::list::List<graphene_std::raster_types::Raster<graphene_std::raster_types::CPU>>>()
.unwrap(); .unwrap();
core_types::record::stack::reserve(executor.tree().stack_need());
let result = edge.eval(&ctx); let result = edge.eval(&ctx);
assert!(matches!(result, GPoll::Final(_)), "the flipped clone must evaluate over record wires, got a non-final poll"); assert!(matches!(result, GPoll::Final(_)), "the flipped clone must evaluate over record wires, got a non-final poll");
} }
@@ -646,6 +648,7 @@ mod test {
.unwrap() .unwrap()
.downcast_record::<graphene_std::list::List<graphene_std::raster::color::Color>>() .downcast_record::<graphene_std::list::List<graphene_std::raster::color::Color>>()
.unwrap(); .unwrap();
core_types::record::stack::reserve(executor.tree().stack_need());
let result = edge.eval(&ctx); let result = edge.eval(&ctx);
assert!(matches!(result, GPoll::Final(_)), "the palette must evaluate through its record wires, got a non-final poll"); assert!(matches!(result, GPoll::Final(_)), "the palette must evaluate through its record wires, got a non-final poll");
} }