diff options
| author | fanquake <fanquake@gmail.com> | 2024-01-05 15:36:57 +0000 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2024-01-05 15:37:06 +0000 |
| commit | 7c248b972b99fd0da084783dfe018c7233323188 (patch) | |
| tree | 863443acf1462a9a488fd17907f0da14da264b64 /src/test | |
| parent | c80f57ba575af96890f185765a53a62ef58ef2c8 (diff) | |
| parent | fa87f8feb76da42eeb5c4d32ee7be070b2bd559f (diff) | |
Merge bitcoin/bitcoin#29042: doc: Clarify C++20 comments
fa87f8feb76da42eeb5c4d32ee7be070b2bd559f doc: Clarify C++20 comments (MarcoFalke)
Pull request description:
Turns out "class template argument deduction for aggregates" is one of the few things implemented only in recent compilers, see https://en.cppreference.com/w/cpp/compiler_support/20
So clarify the comments.
ACKs for top commit:
hebasto:
ACK fa87f8feb76da42eeb5c4d32ee7be070b2bd559f, I verified the code with clang-{16,17}.
Tree-SHA512: f6d20f946cb6f8e34db224e074ed8f9dfa598377c066d1b58a8feb9e64d007444f1e2c0399e91a3e282fd5d59f90e0d7df90aa3956824d96bc78070ee12f603c
Diffstat (limited to 'src/test')
| -rw-r--r-- | src/test/fuzz/fuzz.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/test/fuzz/fuzz.cpp b/src/test/fuzz/fuzz.cpp index 5245b4607b..6de480ff15 100644 --- a/src/test/fuzz/fuzz.cpp +++ b/src/test/fuzz/fuzz.cpp @@ -1,4 +1,4 @@ -// Copyright (c) 2009-2022 The Bitcoin Core developers +// Copyright (c) 2009-present The Bitcoin Core developers // Distributed under the MIT software license, see the accompanying // file COPYING or http://www.opensource.org/licenses/mit-license.php. @@ -71,7 +71,7 @@ auto& FuzzTargets() void FuzzFrameworkRegisterTarget(std::string_view name, TypeTestOneInput target, FuzzTargetOptions opts) { - const auto it_ins{FuzzTargets().try_emplace(name, FuzzTarget /* temporary can be dropped in C++20 */ {std::move(target), std::move(opts)})}; + const auto it_ins{FuzzTargets().try_emplace(name, FuzzTarget /* temporary can be dropped after clang-16 */ {std::move(target), std::move(opts)})}; Assert(it_ins.second); } |