starknet_os: rename standalone adjective variables#12550
starknet_os: rename standalone adjective variables#12550dan-starkware merged 4 commits intomain-v0.14.2from
Conversation
dorimedini-starkware
left a comment
There was a problem hiding this comment.
@dorimedini-starkware reviewed 2 files and all commit messages, and made 1 comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @dan-starkware).
crates/starknet_os/src/hints/hint_implementation/execution/implementation.rs line 747 at r1 (raw file):
let result = ctx.vm.get_range(retdata, min(retdata_size, 100_usize)); let validated_value = MaybeRelocatable::from(ctx.fetch_const(Const::Validated)?);
this name implies that this is the value that has been validated, not the value of the VALIDATED const.
I would keep the previous name, or rename to something like this
Suggestion:
validate_succeeded5a229fc to
48da32e
Compare
95bdc71 to
0b8e14f
Compare
dan-starkware
left a comment
There was a problem hiding this comment.
@dan-starkware made 1 comment.
Reviewable status: 0 of 7 files reviewed, 1 unresolved discussion (waiting on @dorimedini-starkware).
crates/starknet_os/src/hints/hint_implementation/execution/implementation.rs line 747 at r1 (raw file):
Previously, dorimedini-starkware wrote…
this name implies that this is the value that has been validated, not the value of the VALIDATED const.
I would keep the previous name, or rename to something like this
Done.
dorimedini-starkware
left a comment
There was a problem hiding this comment.
@dorimedini-starkware reviewed 7 files and all commit messages, and resolved 1 discussion.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @dan-starkware).

No description provided.