Skip to content

fix: PartitionOptions hash_rate upper bound to match u8 storage #401

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
VolodymyrBg wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: PartitionOptions hash_rate upper bound to match u8 storage #401

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
4 changes: 2 additions & 2 deletions air/src/options.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -189,7 +189,7 @@ impl ProofOptions {
/// # Panics
/// Panics if:
/// - `num_partitions` is zero or greater than 16.
/// - `hash_rate` is zero or greater than 256.
/// - `hash_rate` is zero or greater than 255.
pub const fn with_partitions(
mut self,
num_partitions: usize,
Expand Down Expand Up @@ -414,7 +414,7 @@ impl PartitionOptions {
assert!(num_partitions <= 16, "number of partitions must be smaller than or equal to 16");

assert!(hash_rate >= 1, "hash rate must be greater than or equal to 1");
assert!(hash_rate <= 256, "hash rate must be smaller than or equal to 256");
assert!(hash_rate <= 255, "hash rate must be smaller than or equal to 255");

Self {
num_partitions: num_partitions as u8,
Expand Down