Skip to content
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

async: Add conversions from static slices to Body #203

Merged
merged 1 commit into from Sep 21, 2017

Conversation

Projects
None yet
3 participants
@kamalmarhubi
Copy link
Contributor

kamalmarhubi commented Sep 19, 2017

These useful conversions were missed in #159.

refs #158

@kamalmarhubi

This comment has been minimized.

Copy link
Contributor Author

kamalmarhubi commented Sep 19, 2017

Ah should have added &'static [u8] as well. Will do shortly. Closing for now.

@kamalmarhubi

This comment has been minimized.

Copy link
Contributor Author

kamalmarhubi commented Sep 21, 2017

Added the byte slice version, and avoid copying.

@kamalmarhubi kamalmarhubi reopened this Sep 21, 2017

@kamalmarhubi kamalmarhubi force-pushed the kamalmarhubi:master branch from f0d0473 to 16ef49a Sep 21, 2017

@kamalmarhubi kamalmarhubi changed the title async: Add conversion from static str to Body async: Add conversions from static slices to Body Sep 21, 2017

@KodrAus

This comment has been minimized.

Copy link
Contributor

KodrAus commented Sep 21, 2017

Nice! 👍

async: Add conversions from static slices to Body
These useful conversions were missed in #159.

refs #158

@kamalmarhubi kamalmarhubi force-pushed the kamalmarhubi:master branch from 16ef49a to 87f03e1 Sep 21, 2017

@kamalmarhubi

This comment has been minimized.

Copy link
Contributor Author

kamalmarhubi commented Sep 21, 2017

Rebased.

@seanmonstar seanmonstar merged commit a646bb5 into seanmonstar:master Sep 21, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@seanmonstar

This comment has been minimized.

Copy link
Owner

seanmonstar commented Sep 21, 2017

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.