Skip to content

Conversation

@Anilm3
Copy link
Collaborator

@Anilm3 Anilm3 commented Sep 30, 2025

This PR fixes an issue where single-char matches on the phrase_match operator are discarded.

@Anilm3 Anilm3 requested a review from a team as a code owner September 30, 2025 15:59
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 85.23%. Comparing base (b9b159e) to head (4ff5611).

Files with missing lines Patch % Lines
src/matcher/phrase_match.cpp 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #462   +/-   ##
=======================================
  Coverage   85.23%   85.23%           
=======================================
  Files         186      186           
  Lines        9387     9387           
  Branches     4170     4170           
=======================================
  Hits         8001     8001           
  Misses        552      552           
  Partials      834      834           
Flag Coverage Δ
waf_test 85.23% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Anilm3 Anilm3 enabled auto-merge (squash) September 30, 2025 16:13
@github-actions
Copy link

Artifact Size Comparison 📦

Artifact Previous Release This PR Difference
darwin-arm64::libddwaf.a 89560336 90005656 0.00%
darwin-arm64::libddwaf.a.stripped 4545616 4579560 0.00%
darwin-arm64::libddwaf.dylib 1960784 1963024 0.00%
darwin-universal::libddwaf.a 180532808 181419976 0.00%
darwin-universal::libddwaf.a.stripped 9690824 9765384 0.00%
darwin-universal::libddwaf.dylib 4139856 4142096 0.00%
darwin-x86_64::libddwaf.a 90972424 91414272 0.00%
darwin-x86_64::libddwaf.a.stripped 5145160 5185776 0.00%
darwin-x86_64::libddwaf.dylib 2147776 2150896 0.00%
linux-aarch64::libddwaf.a 72647358 73159494 0.00%
linux-aarch64::libddwaf.a.stripped 11778162 11852226 0.00%
linux-aarch64::libddwaf.so 2453368 2463216 0.00%
linux-armv7::libddwaf.a 64279684 64719098 0.00%
linux-armv7::libddwaf.a.stripped 10783356 10852762 0.00%
linux-armv7::libddwaf.so 2138996 2148652 0.00%
linux-i386::libddwaf.a 62427430 62842618 0.00%
linux-i386::libddwaf.a.stripped 9321362 9379954 0.00%
linux-i386::libddwaf.so 2382908 2392676 0.00%
linux-x86_64::libddwaf.a 73129494 73645934 0.00%
linux-x86_64::libddwaf.a.stripped 11599138 11671234 0.00%
linux-x86_64::libddwaf.so 2649632 2660048 0.00%
windows-arm64::ddwaf.dll 4769280 4788224 0.00%
windows-arm64::ddwaf.lib 11698 11698 0.00%
windows-arm64::ddwaf_static.lib 57431616 57913408 0.00%
windows-win32::ddwaf.dll 3356160 3368448 0.00%
windows-win32::ddwaf.lib 11922 11922 0.00%
windows-win32::ddwaf_static.lib 49113786 49534122 0.00%
windows-x64::ddwaf.dll 4088832 4101632 0.00%
windows-x64::ddwaf.lib 11698 11698 0.00%
windows-x64::ddwaf_static.lib 56948354 57409488 0.00%
windows-x86_64::libddwaf.a 6464848 6539840 0.01%
windows-x86_64::libddwaf.dll 18860246 18890291 0.00%
windows-x86_64::libddwaf.dll.a 31948 31948 0.00%

@pr-commenter
Copy link

pr-commenter bot commented Sep 30, 2025

Benchmarks gcc

Benchmark execution time: 2025-09-30 16:22:19

Comparing candidate commit 4ff5611 in PR branch anilm3/fix-off-by-one-pm with baseline commit b9b159e in branch master.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 1 metrics, 0 unstable metrics.

@pr-commenter
Copy link

pr-commenter bot commented Sep 30, 2025

Benchmarks clang

Benchmark execution time: 2025-09-30 16:22:34

Comparing candidate commit 4ff5611 in PR branch anilm3/fix-off-by-one-pm with baseline commit b9b159e in branch master.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 1 metrics, 0 unstable metrics.

@Anilm3 Anilm3 merged commit 82d66cd into master Sep 30, 2025
56 of 57 checks passed
@Anilm3 Anilm3 deleted the anilm3/fix-off-by-one-pm branch September 30, 2025 16:26
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