Skip to content

Commit fa477d3

Browse files
author
MacroFake
committed
Remove ::GetVirtualTransactionSize() alias
Each alias is only used in one place.
1 parent fa2f6c1 commit fa477d3

File tree

4 files changed

+2
-21
lines changed

4 files changed

+2
-21
lines changed

src/node/psbt.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ PSBTAnalysis AnalyzePSBT(PartiallySignedTransaction psbtx)
137137

138138
if (success) {
139139
CTransaction ctx = CTransaction(mtx);
140-
size_t size = GetVirtualTransactionSize(ctx, GetTransactionSigOpCost(ctx, view, STANDARD_SCRIPT_VERIFY_FLAGS));
140+
size_t size(GetVirtualTransactionSize(ctx, GetTransactionSigOpCost(ctx, view, STANDARD_SCRIPT_VERIFY_FLAGS), ::nBytesPerSigOp));
141141
result.estimated_vsize = size;
142142
// Estimate fee rate
143143
CFeeRate feerate(fee, size);

src/policy/settings.h

-16
Original file line numberDiff line numberDiff line change
@@ -6,22 +6,6 @@
66
#ifndef BITCOIN_POLICY_SETTINGS_H
77
#define BITCOIN_POLICY_SETTINGS_H
88

9-
#include <policy/policy.h>
10-
11-
#include <cstdint>
12-
13-
class CTransaction;
14-
159
extern unsigned int nBytesPerSigOp;
1610

17-
static inline int64_t GetVirtualTransactionSize(int64_t weight, int64_t sigop_cost)
18-
{
19-
return GetVirtualTransactionSize(weight, sigop_cost, ::nBytesPerSigOp);
20-
}
21-
22-
static inline int64_t GetVirtualTransactionSize(const CTransaction& tx, int64_t sigop_cost)
23-
{
24-
return GetVirtualTransactionSize(tx, sigop_cost, ::nBytesPerSigOp);
25-
}
26-
2711
#endif // BITCOIN_POLICY_SETTINGS_H

src/test/fuzz/integer.cpp

-3
Original file line numberDiff line numberDiff line change
@@ -87,9 +87,6 @@ FUZZ_TARGET_INIT(integer, initialize_integer)
8787
}
8888
(void)GetSizeOfCompactSize(u64);
8989
(void)GetSpecialScriptSize(u32);
90-
if (!MultiplicationOverflow(i64, static_cast<int64_t>(::nBytesPerSigOp)) && !AdditionOverflow(i64 * ::nBytesPerSigOp, static_cast<int64_t>(4))) {
91-
(void)GetVirtualTransactionSize(i64, i64);
92-
}
9390
if (!MultiplicationOverflow(i64, static_cast<int64_t>(u32)) && !AdditionOverflow(i64, static_cast<int64_t>(4)) && !AdditionOverflow(i64 * u32, static_cast<int64_t>(4))) {
9491
(void)GetVirtualTransactionSize(i64, i64, u32);
9592
}

src/txmempool.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ void CTxMemPoolEntry::UpdateLockPoints(const LockPoints& lp)
106106

107107
size_t CTxMemPoolEntry::GetTxSize() const
108108
{
109-
return GetVirtualTransactionSize(nTxWeight, sigOpCost);
109+
return GetVirtualTransactionSize(nTxWeight, sigOpCost, ::nBytesPerSigOp);
110110
}
111111

112112
void CTxMemPool::UpdateForDescendants(txiter updateIt, cacheMap& cachedDescendants,

0 commit comments

Comments
 (0)