-
Notifications
You must be signed in to change notification settings - Fork 216
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
imp(Protobuf)!: align
encode{,_length_delimited}_vec
with `prost::Me…
…ssage` to return `Vec<u8>` (#1324) * imp: remove errors for encode{,_length_delimited}_vec methods in Protobuf * chore: add component name + unclog for removing mandatory clones * imp(Protobuf): remove mandatory cloning in default method impls * nit: clippy catches * nit: `to_signable_vec` under `vote.rs` takes self and avoid clone Co-authored-by: Mikhail Zabaluev <mikhail@informal.systems> * nit: `to_signable_vec` under `proposal.rs` takes self and avoid clone Co-authored-by: Mikhail Zabaluev <mikhail@informal.systems> * fix: remove clone under SignProposal/VoteRequest as well --------- Co-authored-by: Mikhail Zabaluev <mikhail@informal.systems>
- Loading branch information
1 parent
63d1859
commit 808835a
Showing
11 changed files
with
58 additions
and
67 deletions.
There are no files selected for viewing
6 changes: 6 additions & 0 deletions
6
.changelog/unreleased/breaking-changes/1323-encode-vec-no-error.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
- `[tendermint-proto]` Align the return signature of the `encode_vec` and | ||
`encode_length_delimited_vec` methods in the `Protobuf` trait with | ||
`prost::Message` by directly returning `Vec<u8>`. | ||
([\#1323](https://github.com/informalsystems/tendermint-rs/issues/1323)) | ||
* Remove mandatory cloning in `Protobuf` methods and let callers decide on | ||
clone beforehand for original value access |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters