diff options
| author | Dan Raviv <[email protected]> | 2017-08-26 16:36:46 +0300 |
|---|---|---|
| committer | Patrick Lodder <[email protected]> | 2021-08-28 21:07:58 +0200 |
| commit | 6efc333ffe4744c325a3b595b26427f144b658c3 (patch) | |
| tree | 79db9105685929e98c740051b87b523ad4135d4e /src/validationinterface.cpp | |
| parent | Merge pull request #2483 from bjacquin/dev/beber/db53 (diff) | |
| download | discoin-6efc333ffe4744c325a3b595b26427f144b658c3.tar.xz discoin-6efc333ffe4744c325a3b595b26427f144b658c3.zip | |
Remove redundant explicitly defined copy ctors
CFeeRate and CTxMemPoolEntry have explicitly defined copy ctors which has
the same functionality as the implicit default copy ctors which would
have been generated otherwise.
Besides being redundant, it violates the rule of three
(see https://en.wikipedia.org/wiki/Rule_of_three_(C%2B%2B_programming) ).
(Of course, the rule of three doesn't -really- cause a resource
management issue here, but the reason for that is exactly that there is
no need for an explicit copy ctor in the first place since no resources
are being managed).
CFeeRate has an explicitly defined copy ctor which has the same
functionality as the implicit default copy ctor which would have been
generated otherwise.
-----------------------------
Cherry-picked from: b426e2467
Note:
Solved conflict where upstream moved CFeeRate into ./policy/
whereas we still have it in amount.h/cpp on 1.14.
Diffstat (limited to 'src/validationinterface.cpp')
0 files changed, 0 insertions, 0 deletions