Skip to content

Conversation

@fiveop
Copy link
Contributor

@fiveop fiveop commented Dec 10, 2016

copy_bytes updates dst so that it points after the bytes that were just
copied into it. encode_into did not advance the buffer in the same way
when encoding the data.

See #473

copy_bytes updates dst so that it points after the bytes that were just
copied into it. encode_into did not advance the buffer in the same way
when encoding the data.
@fiveop
Copy link
Contributor Author

fiveop commented Dec 16, 2016

r? @kamalmarhubi @posborne

@fiveop
Copy link
Contributor Author

fiveop commented Jan 11, 2017

@homu r=fiveop

@homu
Copy link
Contributor

homu commented Jan 11, 2017

📌 Commit 3faaca1 has been approved by fiveop

homu added a commit that referenced this pull request Jan 11, 2017
Fix ControlMessage::encode_into when encoding multiple messages

copy_bytes updates dst so that it points after the bytes that were just
copied into it. encode_into did not advance the buffer in the same way
when encoding the data.

See #473
@homu
Copy link
Contributor

homu commented Jan 11, 2017

⌛ Testing commit 3faaca1 with merge 5d81571...

@homu
Copy link
Contributor

homu commented Jan 11, 2017

☀️ Test successful - status

@homu homu merged commit 3faaca1 into nix-rust:master Jan 11, 2017
@fiveop fiveop deleted the fixcopyinto branch January 18, 2017 08:07
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