Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

chore: fix some comments #2106

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion crates/blockifier/src/execution/entry_point.rs
Original file line number Diff line number Diff line change
Expand Up @@ -210,7 +210,7 @@ impl EntryPointExecutionContext {
}
TransactionInfo::Current(context) => {
// TODO(Ori, 1/2/2024): Write an indicative expect message explaining why the
// convertion works.
// conversion works.
context
.l1_resource_bounds()?
.max_amount
Expand Down
4 changes: 2 additions & 2 deletions crates/blockifier/src/execution/syscalls/syscalls_test.rs
Original file line number Diff line number Diff line change
Expand Up @@ -458,7 +458,7 @@ fn test_get_execution_info(
(
Resource::L1Gas,
// TODO(Ori, 1/2/2024): Write an indicative expect message explaining why the
// convertion works.
// conversion works.
ResourceBounds {
max_amount: max_amount
.0
Expand Down Expand Up @@ -793,7 +793,7 @@ fn test_send_message_to_l1() {
vec![
to_address,
// TODO(Ori, 1/2/2024): Write an indicative expect message explaining why the
// convertion works.
// conversion works.
felt!(u64::try_from(payload.len()).expect("Failed to convert usize to u64.")),
],
payload.clone(),
Expand Down
2 changes: 1 addition & 1 deletion crates/blockifier/src/transaction/account_transaction.rs
Original file line number Diff line number Diff line change
Expand Up @@ -203,7 +203,7 @@ impl AccountTransaction {
return Err(TransactionFeeError::MaxL1GasAmountTooLow {
max_l1_gas_amount,
// TODO(Ori, 1/2/2024): Write an indicative expect message explaining why
// the convertion works.
// the conversion works.
minimal_l1_gas_amount: (minimal_l1_gas_amount
.try_into()
.expect("Failed to convert u128 to u64.")),
Expand Down
4 changes: 2 additions & 2 deletions crates/blockifier/src/versioned_constants.rs
Original file line number Diff line number Diff line change
Expand Up @@ -298,7 +298,7 @@ pub struct OsResources {
// Mapping from every syscall to its execution resources in the OS (e.g., amount of Cairo
// steps).
// TODO(Arni, 14/6/2023): Update `GetBlockHash` values.
// TODO(ilya): Consider moving the resources of a keccak round to a seperate dict.
// TODO(ilya): Consider moving the resources of a keccak round to a separate dict.
execute_syscalls: HashMap<SyscallSelector, ExecutionResources>,
// Mapping from every transaction to its extra execution resources in the OS,
// i.e., resources that don't count during the execution itself.
Expand Down Expand Up @@ -509,7 +509,7 @@ pub struct OsConstants {
}

impl OsConstants {
// List of additinal os constants, beside the gas cost and validate rounding constants, that are
// List of additional os constants, beside the gas cost and validate rounding constants, that are
// not used by the blockifier but included for transparency. These constanst will be ignored
// during the creation of the struct containing the gas costs.

Expand Down
Loading