Skip to content

IoTask<Ts...> instead of Task<std::tuple<std::error_code, Ts....>> - #38

Merged
pgit merged 3 commits into
masterfrom
io-task
Nov 6, 2026
Merged

pgit merged 3 commits into
masterfrom
io-task

Conversation

@pgit

@pgit pgit commented Nov 6, 2026

Copy link
Copy Markdown
Owner

No description provided.

pgit and others added 3 commits November 6, 2026 17:48
…Task<Ts...>

Every operation yields its error first, so std::tuple<error_code, ...> was spelled out ~100
times. results<Ts...> is capy's alias with the runtime's error_code (boost::system's on ASIO,
capy::results itself on COROSIO), and IoTask<Ts...> is Task<results<Ts...>>. Task<T> stays the
plain coroutine, so Task<void> and value tasks are unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Breathe showed each type of the runtime layer as what it is on ASIO, and each code of errors::
with its ASIO initializer. The types are now declared in reference.rst without a definition,
with what each is on ASIO and on COROSIO. The errors stay with Doxygen, as comments elsewhere
link to them, with \hideinitializer and a \runtimes{asio, corosio} alias that renders the same
two lines.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Re-enable modernize-concat-nested-namespaces and performance-avoid-endl,
which had one and three findings. Disable readability-redundant-qualified-alias
(aliases read better than using-declarations, in the runtime headers and in
Doxygen) and performance-enum-size. Mark the intended implicit conversions
NOLINT, drop unused includes and a redundant typename.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@pgit
pgit merged commit b489baa into master Nov 6, 2026
1 of 11 checks passed
@pgit
pgit deleted the io-task branch November 6, 2026 20:17
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.

1 participant