crypto: more Span<std::byte> modernization & follow-ups #43
rajarshimaitra
started this conversation in
PR Reviews
Replies: 2 comments
-
Summary
Questions
|
Beta Was this translation helpful? Give feedback.
0 replies
-
Summary1. Did you review the PR? Concept ACK, approach ACK, tested ACK, or NACK? What was your review approach?Concept/tested ACK by the participants who went through the code. 2. What is ChaCha20 encryption algorithm? Why are there 2 interfaces - ChaCha20 and ChaCha20Aligned? (hint: Check comments on src/crypto/chacha20.h)
3. Can you pick out some examples of modern c++ code used in this PR? what's the advantage of using these in ChaCha20?(hint: auto, span, std::byte, noexcept, std::array, nodiscard, template, optional, constexpr. You can search up C++ features in https://en.cppreference.com/ and get the C++ version in which it was introduced/updates.)
4. Which version of C++ does bitcoin core support? What's the disadvantage of updating to newer C++ versions?We're currently in C++17. It takes time for C++ version to be available on all common platform/linux distributions. We don't generally update it till it's stable + commonly available. For example, Backports like this can take time to be commonly available. 5. What does
|
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Session Details
Span<std::byte>
modernization & follow-ups bitcoin/bitcoin#28100[Crypto][C++]
[c++]
Learning
Beta Was this translation helpful? Give feedback.
All reactions