Skip to content

Conversation

@atuchin-m
Copy link
Collaborator

@atuchin-m atuchin-m commented Nov 11, 2025

The PR addresses matching issues that makes the behavior unstable: the result depends on the order we checked the tokens.

@atuchin-m atuchin-m self-assigned this Nov 11, 2025
@atuchin-m atuchin-m requested a review from a team as a code owner November 11, 2025 20:05
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Rust Benchmark

Benchmark suite Current: cd3df6a Previous: 2e8522e Ratio
rule-match-browserlike/brave-list 1952435059 ns/iter (± 6995158) 2018318447 ns/iter (± 21765134) 0.97
rule-match-first-request/brave-list 1086932 ns/iter (± 10810) 1125936 ns/iter (± 7201) 0.97
blocker_new/brave-list 151067384 ns/iter (± 1023001) 152949922 ns/iter (± 796277) 0.99
blocker_new/brave-list-deserialize 26575134 ns/iter (± 153793) 26230664 ns/iter (± 2358387) 1.01
memory-usage/brave-list-initial 10212216 ns/iter (± 3) 10212216 ns/iter (± 3) 1
memory-usage/brave-list-initial/max 63149135 ns/iter (± 3) 63149135 ns/iter (± 3) 1
memory-usage/brave-list-initial/alloc-count 1231721 ns/iter (± 3) 1231721 ns/iter (± 3) 1
memory-usage/brave-list-1000-requests 2645938 ns/iter (± 3) 2666990 ns/iter (± 3) 0.99
memory-usage/brave-list-1000-requests/alloc-count 71016 ns/iter (± 3) 71425 ns/iter (± 3) 0.99
url_cosmetic_resources/brave-list 190034 ns/iter (± 767) 189892 ns/iter (± 986) 1.00
cosmetic-class-id-match/brave-list 3560017 ns/iter (± 1023323) 3396414 ns/iter (± 922069) 1.05

This comment was automatically generated by workflow using github-action-benchmark.

@atuchin-m atuchin-m merged commit e9299eb into master Nov 12, 2025
9 checks passed
@atuchin-m atuchin-m deleted the pr-fix-no-hostname-case branch November 12, 2025 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants