Skip to content

Improve std adapters, fix problems triggered in amalgamate.py#156

Merged
biojppm merged 6 commits intomasterfrom
fix/amalgamate
Mar 13, 2026
Merged

Improve std adapters, fix problems triggered in amalgamate.py#156
biojppm merged 6 commits intomasterfrom
fix/amalgamate

Conversation

@biojppm
Copy link
Owner

@biojppm biojppm commented Mar 13, 2026

  • Add traits class c4::is_writeable_string<>
  • c4/std/*.hpp adapters: add missing forward declarations of c4::is_string<>
  • Add default-less csubstr::copy_from() to simplify implementations of to_chars()/from_chars()
  • Minor internal changes in c4::substr
  • Add C++20 std::span<char> and std::span<const char> adapters in new header c4/std/span.hpp

@codecov
Copy link

codecov bot commented Mar 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.91%. Comparing base (68de1b6) to head (5664ef3).
⚠️ Report is 7 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #156      +/-   ##
==========================================
- Coverage   96.91%   96.91%   -0.01%     
==========================================
  Files          28       28              
  Lines        3149     3141       -8     
==========================================
- Hits         3052     3044       -8     
  Misses         97       97              

☔ 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.

@coveralls
Copy link

coveralls commented Mar 13, 2026

Pull Request Test Coverage Report for Build 23031680485

Details

  • 23 of 23 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.005%) to 96.857%

Totals Coverage Status
Change from base Build 22970124683: -0.005%
Covered Lines: 3113
Relevant Lines: 3214

💛 - Coveralls

@biojppm biojppm changed the title Fix amalgamate Improve std adapters, fix problems triggered in amalgamate.py Mar 13, 2026
@biojppm biojppm merged commit 07009bb into master Mar 13, 2026
298 of 299 checks passed
@biojppm biojppm deleted the fix/amalgamate branch March 13, 2026 12:21
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.

2 participants