Refactor: Removed begin/end_ptr functions.#9305
Merged
laanwj merged 1 commit intobitcoin:masterfrom Dec 13, 2016
Merged
Conversation
Member
|
Concept ACK |
dcousens
approved these changes
Dec 9, 2016
Contributor
|
ACK 8c1dbc5 |
Member
|
I was about to mention the developer notes, but they're already gone from there. utACK 8c1dbc5 |
laanwj
added a commit
that referenced
this pull request
Dec 13, 2016
8c1dbc5 Refactor: Removed begin/end_ptr functions. (Karl-Johan Alm)
codablock
pushed a commit
to codablock/dash
that referenced
this pull request
Jan 17, 2018
8c1dbc5 Refactor: Removed begin/end_ptr functions. (Karl-Johan Alm)
andvgal
pushed a commit
to energicryptocurrency/gen2-energi
that referenced
this pull request
Jan 6, 2019
8c1dbc5 Refactor: Removed begin/end_ptr functions. (Karl-Johan Alm)
CryptoCentric
pushed a commit
to absolute-community/absolute
that referenced
this pull request
Feb 25, 2019
8c1dbc5 Refactor: Removed begin/end_ptr functions. (Karl-Johan Alm)
furszy
added a commit
to PIVX-Project/PIVX
that referenced
this pull request
Apr 8, 2021
172fe15 Add missing locks and locking annotations for CAddrMan (practicalswift) 9271ace Support serialization as another type without casting (Pieter Wuille) dc37dd9 Remove unnecessary NONNEGATIVE_SIGNED (Russell Yanofsky) ddd2ab1 Add static_assert to prevent VARINT(<signed value>) (Russell Yanofsky) 539db35 Support deserializing into temporaries (Pieter Wuille) 36db7fd Merge READWRITEMANY into READWRITE (Pieter Wuille) 08ebd5b Remove old pre C++11 functions begin_ptr/end_ptr. (furszy) 9c84665 Prevent integer overflow in ReadVarInt. (Gregory Maxwell) Pull request description: Back ported some pretty concise PRs from upstream over the data serialization area. * bitcoin#9305. --> removal of pre c++11 compatibility functions. * bitcoin#9693. --> prevent integer overflow in `ReadVarInt`. * bitcoin#9753. --> compile error if VARINT is called with a signed value. * bitcoin#12683. --> fixing constness violations * bitcoin#12731. --> support for `READWRITEAS` macro. ACKs for top commit: random-zebra: ACK 172fe15 Fuzzbawls: ACK 172fe15 Tree-SHA512: 1e1e697761b885dcc1aed8a2132bed693b1c76f1f2ed22ae5c074dfb4c353b81d307f71a4c12ed71fc39fd2207c1403881bd699e32b85a167bee57b4f0946130
This file contains hidden or 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
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
serialize.hhas a number of functions for accessing the data insidestd::vectors, prepended with a warning to not use them in new code, as "with C++11 the language has built-in functionality for this and it's more readable to just use that". This PR removes these functions and their uses.begin_ptr(X)→X.data()end_ptr(X)→(X.data() + X.size())