Skip to content

Commit

Permalink
Re-word: it's timing per request, not per call to SNFE.match()
Browse files Browse the repository at this point in the history
A single request can require multiple calls to the static network
filtering engine (SNFE). The reported timing is the result of
going through *all* the required calls to SNFE. In effect, a
single call to SNFE.match() is a fraction of the reported timing.
  • Loading branch information
gorhill committed Oct 7, 2024
1 parent a80c749 commit ec3852b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/js/benchmarks.js
Original file line number Diff line number Diff line change
Expand Up @@ -217,7 +217,7 @@ export async function benchmarkStaticNetFiltering(options = {}) {

const output = [
'Benchmarked static network filtering engine:',
`\tEvaluated ${matchCount} match calls in ${dur.toFixed(0)} ms`,
`\tEvaluated ${matchCount} requests in ${dur.toFixed(0)} ms`,
`\tAverage: ${(dur / matchCount).toFixed(3)} ms per request`,
`\tNot blocked: ${matchCount - blockCount - allowCount}`,
`\tBlocked: ${blockCount}`,
Expand Down

0 comments on commit ec3852b

Please sign in to comment.