-
Notifications
You must be signed in to change notification settings - Fork 925
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ArrayPtr-based async read api. #2084
base: v2
Are you sure you want to change the base?
Conversation
c5dab09
to
7cae47a
Compare
Downstream of capnproto/capnproto#2084
f5325ec
to
4f73201
Compare
Downstream of capnproto/capnproto#2084
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are there any parts of this PR that aren't just mechanical updates which we should pay particular attention to?
@@ -1514,7 +1514,7 @@ public: | |||
co_return true; | |||
} | |||
|
|||
auto amount = co_await inner.tryRead(headerBuffer.begin(), 1, headerBuffer.size()); | |||
auto amount = co_await inner.tryRead(headerBuffer.asBytes(), 1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
redundant asBytes()
?
Not really. I don't think I have changed anything in "real" code. Tests had some cleanup that was mostly related to string/bytes simplifications where possible. Otherwise I tried to stay as close as possible to the original. |
I also have downstream changes ready and I obviously plan to make sure all builds pass everywhere before starting to merge. |
4f73201
to
d04996c
Compare
Downstream of capnproto/capnproto#2084
No description provided.