Skip to content
This repository was archived by the owner on Feb 1, 2024. It is now read-only.

Commit 5db6192

Browse files
committed
Fix manual_unwrap_or warnings
Signed-off-by: Joseph Livesey <joseph.livesey@btp.works>
1 parent 4a50c85 commit 5db6192

File tree

2 files changed

+25
-32
lines changed

2 files changed

+25
-32
lines changed

perf/sawtooth_workload/src/main.rs

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -109,10 +109,11 @@ fn create_batch_subcommand_args<'a, 'b>() -> App<'a, 'b> {
109109
}
110110

111111
fn run_batch_command(args: &ArgMatches) -> Result<(), Box<dyn Error>> {
112-
let max_txns: usize = match args.value_of("max-batch-size").unwrap_or("100").parse() {
113-
Ok(n) => n,
114-
Err(_) => 0,
115-
};
112+
let max_txns: usize = args
113+
.value_of("max-batch-size")
114+
.unwrap_or("100")
115+
.parse()
116+
.unwrap_or(0);
116117

117118
if max_txns == 0 {
118119
return arg_error("max-batch-size must be a number greater than 0");
@@ -175,10 +176,7 @@ fn create_submit_subcommand_args<'a, 'b>() -> App<'a, 'b> {
175176
}
176177

177178
fn run_submit_command(args: &ArgMatches) -> Result<(), Box<dyn Error>> {
178-
let rate: usize = match args.value_of("rate").unwrap_or("1").parse() {
179-
Ok(n) => n,
180-
Err(_) => 0,
181-
};
179+
let rate: usize = args.value_of("rate").unwrap_or("1").parse().unwrap_or(0);
182180

183181
if rate == 0 {
184182
return arg_error("rate must be a number greater than 0");

perf/smallbank_workload/src/main.rs

Lines changed: 19 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -156,17 +156,20 @@ fn create_load_subcommand_args<'a, 'b>() -> App<'a, 'b> {
156156
}
157157

158158
fn run_load_command(args: &ArgMatches) -> Result<(), Box<dyn Error>> {
159-
let max_txns: usize = match args.value_of("max-batch-size").unwrap_or("1").parse() {
160-
Ok(n) => n,
161-
Err(_) => 0,
162-
};
159+
let max_txns: usize = args
160+
.value_of("max-batch-size")
161+
.unwrap_or("1")
162+
.parse()
163+
.unwrap_or(0);
164+
163165
if max_txns == 0 {
164166
return arg_error("max-batch-size must be a number greater than 0");
165167
}
166-
let accounts: usize = match args.value_of("num-accounts").unwrap_or("100").parse() {
167-
Ok(n) => n,
168-
Err(_) => 100,
169-
};
168+
let accounts: usize = args
169+
.value_of("num-accounts")
170+
.unwrap_or("100")
171+
.parse()
172+
.unwrap_or(100);
170173
if accounts == 0 {
171174
return arg_error("The number of accounts must be greater than 0.");
172175
}
@@ -296,10 +299,11 @@ fn create_batch_subcommand_args<'a, 'b>() -> App<'a, 'b> {
296299
}
297300

298301
fn run_batch_command(args: &ArgMatches) -> Result<(), Box<dyn Error>> {
299-
let max_txns: usize = match args.value_of("max-batch-size").unwrap_or("100").parse() {
300-
Ok(n) => n,
301-
Err(_) => 0,
302-
};
302+
let max_txns: usize = args
303+
.value_of("max-batch-size")
304+
.unwrap_or("100")
305+
.parse()
306+
.unwrap_or(0);
303307

304308
if max_txns == 0 {
305309
return arg_error("max-batch-size must be a number greater than 0");
@@ -362,10 +366,7 @@ fn create_submit_subcommand_args<'a, 'b>() -> App<'a, 'b> {
362366
}
363367

364368
fn run_submit_command(args: &ArgMatches) -> Result<(), Box<dyn Error>> {
365-
let rate: usize = match args.value_of("rate").unwrap_or("1").parse() {
366-
Ok(n) => n,
367-
Err(_) => 0,
368-
};
369+
let rate: usize = args.value_of("rate").unwrap_or("1").parse().unwrap_or(0);
369370

370371
if rate == 0 {
371372
return arg_error("rate must be a number greater than 0");
@@ -495,19 +496,13 @@ fn run_playlist_command(args: &ArgMatches) -> Result<(), Box<dyn Error>> {
495496
}
496497

497498
fn run_playlist_create_command(args: &ArgMatches) -> Result<(), Box<dyn Error>> {
498-
let num_accounts = match args.value_of("accounts").unwrap().parse() {
499-
Ok(n) => n,
500-
Err(_) => 0,
501-
};
499+
let num_accounts = args.value_of("accounts").unwrap().parse().unwrap_or(0);
502500

503501
if num_accounts < 2 {
504502
return arg_error("'accounts' must be a number greater than 2");
505503
}
506504

507-
let num_transactions = match args.value_of("transactions").unwrap().parse() {
508-
Ok(n) => n,
509-
Err(_) => 0,
510-
};
505+
let num_transactions = args.value_of("transactions").unwrap().parse().unwrap_or(0);
511506

512507
if num_transactions == 0 {
513508
return arg_error("'transactions' must be a number greater than 0");

0 commit comments

Comments
 (0)