Skip to content

Commit 17e21bc

Browse files
committed
review: use TransactionScript::from_library and
`NoteScript::from_library`
1 parent a5751fb commit 17e21bc

1 file changed

Lines changed: 8 additions & 12 deletions

File tree

  • crates/miden-standards/src/code_builder

crates/miden-standards/src/code_builder/mod.rs

Lines changed: 8 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -368,16 +368,14 @@ impl CodeBuilder {
368368
)
369369
})?;
370370

371-
TransactionScript::from_library(&Self::apply_advice_map_to_library(
372-
advice_map,
373-
Arc::unwrap_or_clone(tx_script_lib),
374-
))
375-
.map_err(|err| {
371+
let tx_script = TransactionScript::from_library(&tx_script_lib).map_err(|err| {
376372
CodeBuilderError::build_error_with_source(
377373
"failed to create transaction script from library",
378374
err,
379375
)
380-
})
376+
})?;
377+
378+
Ok(tx_script.with_advice_map(advice_map))
381379
}
382380

383381
/// Compiles the provided MASM code into a [`NoteScript`].
@@ -398,16 +396,14 @@ impl CodeBuilder {
398396
CodeBuilderError::build_error_with_report("failed to parse note script library", err)
399397
})?;
400398

401-
NoteScript::from_library(&Self::apply_advice_map_to_library(
402-
advice_map,
403-
Arc::unwrap_or_clone(note_script_lib),
404-
))
405-
.map_err(|err| {
399+
let note_script = NoteScript::from_library(&note_script_lib).map_err(|err| {
406400
CodeBuilderError::build_error_with_source(
407401
"failed to create note script from library",
408402
err,
409403
)
410-
})
404+
})?;
405+
406+
Ok(note_script.with_advice_map(advice_map))
411407
}
412408

413409
// ACCESSORS

0 commit comments

Comments
 (0)