Skip to content

Feature/any write stream readiness - #394

Merged
sgerbino merged 2 commits into
cppalliance:developfrom
sgerbino:feature/any-write-stream-readiness
Aug 31, 2026
Merged

Feature/any write stream readiness#394
sgerbino merged 2 commits into
cppalliance:developfrom
sgerbino:feature/any-write-stream-readiness

Conversation

@sgerbino

Copy link
Copy Markdown
Collaborator

No description provided.

…eady

Construct the concrete awaitable in the wrapper's await_ready and
forward its readiness, matching any_read_stream. A stream whose
awaitable reports ready now completes the erased write without
suspending the coroutine, which is the behavior the class docs
already promised. An empty write still never touches the underlying
stream, and pending streams suspend through await_suspend as before.

Claude-Session: https://claude.ai/code/session_01F1KRdVbs1AubbqDNX3KkwD
The erased read wrapper has constructed the concrete awaitable in
await_ready and forwarded its readiness since the beginning, but no
test asserted it: every shared mock reports not-ready, so reverting
to suspend-first would have passed the suite. Mirror the write-side
readiness test so both wrappers' suspension skip is pinned.

Claude-Session: https://claude.ai/code/session_01F1KRdVbs1AubbqDNX3KkwD
@cppalliance-bot

Copy link
Copy Markdown

An automated preview of the documentation is available at https://394.capy.prtest3.cppalliance.org/index.html

If more commits are pushed to the pull request, the docs will rebuild at the same URL.

2026-08-31 13:08:25 UTC

@codecov

codecov Bot commented Aug 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.09%. Comparing base (ee317c4) to head (e3de715).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #394      +/-   ##
===========================================
- Coverage    98.09%   98.09%   -0.01%     
===========================================
  Files          130      130              
  Lines         6291     6289       -2     
===========================================
- Hits          6171     6169       -2     
  Misses         120      120              
Flag Coverage Δ
linux 98.08% <ø> (-0.01%) ⬇️

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

Files with missing lines Coverage Δ
include/boost/capy/io/any_write_stream.hpp 100.00% <100.00%> (ø)

... and 3 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ee317c4...e3de715. Read the comment docs.

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

@cppalliance-bot

Copy link
Copy Markdown

GCOVR code coverage report https://394.capy.prtest3.cppalliance.org/gcovr/index.html
LCOV code coverage report https://394.capy.prtest3.cppalliance.org/genhtml/index.html
Coverage Diff Report https://394.capy.prtest3.cppalliance.org/diff-report/index.html

Build time: 2026-08-31 13:23:32 UTC

@sgerbino
sgerbino merged commit 9dcd6b7 into cppalliance:develop Aug 31, 2026
39 checks passed
@sgerbino
sgerbino deleted the feature/any-write-stream-readiness branch August 31, 2026 13:36
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Beast2 Aug 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants