Skip to content

Commit

Permalink
Fix nightly clippy warning
Browse files Browse the repository at this point in the history
  • Loading branch information
@fornwall
fornwall committed Dec 12, 2024
1 parent 03bfcf1 commit 610dd7b
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions crates/core/src/year2021/day19.rs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -220,10 +220,10 @@ fn find_match(matched_scans: &[Scan], scan_to_join: &Scan) -> Option<(Point, Sca
matched_scan
.fingerprints
.get(to_join_fingerprint)
.map_or(false, |matched_pairs| {
num_pairs_matching_fingerprint += to_join_pairs.len() * matched_pairs.len();
true
.inspect(|matched_pairs| {
num_pairs_matching_fingerprint += to_join_pairs.len() * matched_pairs.len()
})
.is_some()
})
.collect::<Vec<_>>();

Expand Down

1 comment on commit 610dd7b

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more. 

@@                     Benchmark Difference                     @@
#      Name   Old (instructions)   New (instructions)   Change (%)
  2024_13_1           10,000,000           10,000,000            0
  2024_13_2           10,000,000           10,000,000            0
  2024_14_1           10,000,000           10,000,000            0
  2024_14_2           10,000,000           10,000,000            0
  2024_15_1           10,000,000           10,000,000            0
  2024_15_2           10,000,000           10,000,000            0
  2024_16_1           10,000,000           10,000,000            0
  2024_16_2           10,000,000           10,000,000            0
  2024_17_1           10,000,000           10,000,000            0
  2024_17_2           10,000,000           10,000,000            0
  2024_18_1           10,000,000           10,000,000            0
  2024_18_2           10,000,000           10,000,000            0
  2024_19_1           10,000,000           10,000,000            0
  2024_19_2           10,000,000           10,000,000            0
  2024_20_1           10,000,000           10,000,000            0
  2024_20_2           10,000,000           10,000,000            0
  2024_21_1           10,000,000           10,000,000            0
  2024_21_2           10,000,000           10,000,000            0
  2024_22_1           10,000,000           10,000,000            0
  2024_22_2           10,000,000           10,000,000            0
  2024_23_1           10,000,000           10,000,000            0
  2024_23_2           10,000,000           10,000,000            0
  2024_24_1           10,000,000           10,000,000            0
  2024_24_2           10,000,000           10,000,000            0
  2024_25_1           10,000,000           10,000,000            0
   2024_1_1            1,041,059            1,041,059            0
   2024_1_2            1,070,151            1,070,151            0
   2024_2_1            1,641,475            1,641,475            0
   2024_2_2            1,863,553            1,863,553            0
   2024_3_1              227,351              227,351            0
   2024_3_2              317,723              317,723            0
   2024_4_1            1,213,742            1,213,742            0
   2024_4_2              453,872              453,872            0
   2024_5_1            1,605,017            1,605,017            0
   2024_5_2            1,718,959            1,718,959            0
   2024_6_1              641,425              641,425            0
   2024_6_2          141,152,033          141,152,033            0
   2024_7_1            2,466,865            2,466,865            0
   2024_7_2            3,758,374            3,758,374            0
   2024_8_1               44,379               44,379            0
   2024_8_2               86,836               86,836            0
   2024_9_1              897,512              897,512            0
   2024_9_2            2,497,024            2,497,024            0
  2024_10_1            1,087,459            1,087,459            0
  2024_10_2            1,089,358            1,089,358            0
  2024_11_1              959,639              959,844            0
  2024_11_2           36,198,040           36,198,308            0
  2024_12_1            7,663,665            7,663,665            0
  2024_12_2            7,942,216            7,942,216            0
Benchmark Instructions (count) Instructions (%)
2024_6_2 141,152,033 30.2
2024_11_2 36,198,308 7.7
2024_25_1 10,000,000 2.1
2024_24_2 10,000,000 2.1
2024_24_1 10,000,000 2.1
2024_23_2 10,000,000 2.1
2024_23_1 10,000,000 2.1
2024_22_2 10,000,000 2.1
2024_22_1 10,000,000 2.1
2024_21_2 10,000,000 2.1
2024_21_1 10,000,000 2.1
2024_20_2 10,000,000 2.1
2024_20_1 10,000,000 2.1
2024_19_2 10,000,000 2.1
2024_19_1 10,000,000 2.1
2024_18_2 10,000,000 2.1
2024_18_1 10,000,000 2.1
2024_17_2 10,000,000 2.1
2024_17_1 10,000,000 2.1
2024_16_2 10,000,000 2.1
2024_16_1 10,000,000 2.1
2024_15_2 10,000,000 2.1
2024_15_1 10,000,000 2.1
2024_14_2 10,000,000 2.1
2024_14_1 10,000,000 2.1
2024_13_2 10,000,000 2.1
2024_13_1 10,000,000 2.1
2024_12_2 7,942,216 1.7
2024_12_1 7,663,665 1.6
2024_7_2 3,758,374 0.8
2024_9_2 2,497,024 0.5
2024_7_1 2,466,865 0.5
2024_2_2 1,863,553 0.4
2024_5_2 1,718,959 0.4
2024_2_1 1,641,475 0.4
2024_5_1 1,605,017 0.3
2024_4_1 1,213,742 0.3
2024_10_2 1,089,358 0.2
2024_10_1 1,087,459 0.2
2024_1_2 1,070,151 0.2
2024_1_1 1,041,059 0.2
2024_11_1 959,844 0.2
2024_9_1 897,512 0.2
2024_6_1 641,425 0.1
2024_4_2 453,872 0.1
2024_3_2 317,723 0.1
2024_3_1 227,351 0.0
2024_8_2 86,836 0.0
2024_8_1 44,379 0.0

Please sign in to comment.