Skip to content

Fix bad has_from_chars concept#497

Merged
cataphract merged 1 commit into
masterfrom
glopes/has_from_chars
Jun 12, 2026
Merged

Fix bad has_from_chars concept#497
cataphract merged 1 commit into
masterfrom
glopes/has_from_chars

Conversation

@cataphract

Copy link
Copy Markdown
Contributor

from_chars takes the value as a (non-const) reference, so the rvalue reference produced by declval doesn't bind.

@cataphract
cataphract requested a review from a team as a code owner June 12, 2026 15:23
@cataphract
cataphract enabled auto-merge (squash) June 12, 2026 15:30
@datadog-datadog-prod-us1

Copy link
Copy Markdown

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 84.79% (+0.00%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 80dbf16 | Docs | Datadog PR Page | Give us feedback!

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.78%. Comparing base (c5fdbb3) to head (80dbf16).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #497   +/-   ##
=======================================
  Coverage   84.78%   84.78%           
=======================================
  Files         191      191           
  Lines        9341     9341           
  Branches     4184     4184           
=======================================
  Hits         7920     7920           
  Misses        529      529           
  Partials      892      892           
Flag Coverage Δ
waf_test 84.78% <ø> (ø)

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

☔ View full report in Codecov by Harness.
📢 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.

@github-actions

Copy link
Copy Markdown

Dynamic Artifact Size Comparison 📦

Artifact Previous Release This PR Difference
darwin-arm64::libddwaf.dylib 2020576 2020032 -0.02%
darwin-universal::libddwaf.dylib 4248800 4231872 -0.39%
darwin-x86_64::libddwaf.dylib 2208040 2207496 -0.02%
linux-aarch64::libddwaf.so 2388576 2385792 -0.11%
linux-armv7::libddwaf.so 2056400 2055488 -0.04%
linux-i386::libddwaf.so 2312900 2306036 -0.29%
linux-x86_64::libddwaf.so 2581376 2578080 -0.12%
windows-arm64::ddwaf.dll 6731264 6718976 -0.18%
windows-win32::ddwaf.dll 3330560 3322368 -0.24%
windows-x64::ddwaf.dll 4061184 4062208 0.02%

Static Artifact Size Comparison 📦

Artifact Previous Release This PR Difference
darwin-arm64::libddwaf.a 92531240 92529800 0.00%
darwin-arm64::libddwaf.a.stripped 4715920 4716264 0.00%
darwin-universal::libddwaf.a 186577856 186575024 0.00%
darwin-universal::libddwaf.a.stripped 10041896 10042896 0.00%
darwin-x86_64::libddwaf.a 94046568 94045176 0.00%
darwin-x86_64::libddwaf.a.stripped 5325928 5326584 0.01%
linux-aarch64::libddwaf.a 75455020 75467390 0.01%
linux-aarch64::libddwaf.a.stripped 12184592 12182450 -0.01%
linux-armv7::libddwaf.a 66674258 66685682 0.01%
linux-armv7::libddwaf.a.stripped 11189582 11189774 0.00%
linux-i386::libddwaf.a 64858564 64868742 0.01%
linux-i386::libddwaf.a.stripped 9676948 9676154 0.00%
linux-x86_64::libddwaf.a 75942542 75952106 0.01%
linux-x86_64::libddwaf.a.stripped 11985466 11985126 0.00%
windows-arm64::ddwaf.lib 16902 16902 0.00%
windows-arm64::ddwaf_static.lib 135254130 134609214 -0.47%
windows-win32::ddwaf.lib 17228 17228 0.00%
windows-win32::ddwaf_static.lib 47779818 47460088 -0.66%
windows-x64::ddwaf.lib 16902 16902 0.00%
windows-x64::ddwaf_static.lib 55616486 55237786 -0.68%

@cataphract
cataphract merged commit 7a17b8d into master Jun 12, 2026
56 of 57 checks passed
@cataphract
cataphract deleted the glopes/has_from_chars branch June 12, 2026 15:52
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.

3 participants