Skip to content

Commit e0358cc

Browse files
committed
Clippy fixes
1 parent 669cebc commit e0358cc

File tree

3 files changed

+9
-9
lines changed

3 files changed

+9
-9
lines changed

src/queue/mysql.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1231,7 +1231,7 @@ impl DatabaseQueue for crate::queue::JobQueue<MySql> {
12311231
_ => crate::priority::JobPriority::Normal,
12321232
};
12331233

1234-
priority_times.entry(priority).or_insert_with(Vec::new).push(processing_ms as f64);
1234+
priority_times.entry(priority).or_default().push(processing_ms as f64);
12351235
}
12361236

12371237
// Calculate average processing times for each priority

src/queue/postgres.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1353,7 +1353,7 @@ impl DatabaseQueue for crate::queue::JobQueue<Postgres> {
13531353
_ => crate::priority::JobPriority::Normal,
13541354
};
13551355

1356-
priority_times.entry(priority).or_insert_with(Vec::new).push(processing_ms as f64);
1356+
priority_times.entry(priority).or_default().push(processing_ms as f64);
13571357
}
13581358

13591359
// Calculate average processing times for each priority

src/queue/test.rs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1454,10 +1454,10 @@ impl DatabaseQueue for TestQueue {
14541454
crate::priority::JobPriority::High,
14551455
crate::priority::JobPriority::Critical,
14561456
] {
1457-
job_counts.insert(priority.clone(), 0);
1458-
avg_processing_times.insert(priority.clone(), 0.0);
1459-
recent_throughput.insert(priority.clone(), 0);
1460-
priority_distribution.insert(priority.clone(), 0.0);
1457+
job_counts.insert(priority, 0);
1458+
avg_processing_times.insert(priority, 0.0);
1459+
recent_throughput.insert(priority, 0);
1460+
priority_distribution.insert(priority, 0.0);
14611461
}
14621462

14631463
let mut total_jobs = 0u64;
@@ -1466,7 +1466,7 @@ impl DatabaseQueue for TestQueue {
14661466
for jobs in queue_jobs.values() {
14671467
for job_id in jobs {
14681468
if let Some(job) = storage.jobs.get(job_id) {
1469-
*job_counts.entry(job.priority.clone()).or_insert(0) += 1;
1469+
*job_counts.entry(job.priority).or_insert(0) += 1;
14701470
total_jobs += 1;
14711471

14721472
// Calculate processing time if job is completed
@@ -1475,7 +1475,7 @@ impl DatabaseQueue for TestQueue {
14751475
let current_avg = avg_processing_times.get(&job.priority).unwrap_or(&0.0);
14761476
let current_count = *job_counts.get(&job.priority).unwrap_or(&1);
14771477
let new_avg = (current_avg * (current_count - 1) as f64 + processing_time) / current_count as f64;
1478-
avg_processing_times.insert(job.priority.clone(), new_avg);
1478+
avg_processing_times.insert(job.priority, new_avg);
14791479
}
14801480
}
14811481
}
@@ -1489,7 +1489,7 @@ impl DatabaseQueue for TestQueue {
14891489
} else {
14901490
0.0
14911491
};
1492-
priority_distribution.insert(priority.clone(), percentage);
1492+
priority_distribution.insert(*priority, percentage);
14931493
}
14941494

14951495
Ok(crate::priority::PriorityStats {

0 commit comments

Comments
 (0)