DragonX/dragonx
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 2 Wiki Activity

Use max priority for all shielded transfers

Browse Source
This commit is contained in:
Eirik Ogilvie-Wigley
2018-09-28 10:32:30 -06:00
parent e6f6c5d08c
commit a13492744d
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/coins.cpp
View File

@@ -616,13 +616,13 @@ double CCoinsViewCache::GetPriority(const CTransaction &tx, int nHeight) const
if (tx.IsCoinBase()) if (tx.IsCoinBase())
return 0.0; return 0.0;
// Joinsplits do not reveal any information about the value or age of a note, so we // Shielded transfers do not reveal any information about the value or age of a note, so we
// cannot apply the priority algorithm used for transparent utxos. Instead, we just // cannot apply the priority algorithm used for transparent utxos. Instead, we just
// use the maximum priority whenever a transaction contains any JoinSplits. // use the maximum priority for all (partially or fully) shielded transactions.
// (Note that coinbase transactions cannot contain JoinSplits.) // (Note that coinbase transactions cannot contain JoinSplits, or Sapling shielded Spends or Outputs.)
// FIXME: this logic is partially duplicated between here and CreateNewBlock in miner.cpp. // FIXME: this logic is partially duplicated between here and CreateNewBlock in miner.cpp.
if (tx.vjoinsplit.size() > 0) { if (tx.vjoinsplit.size() > 0 || tx.vShieldedSpend.size() > 0 || tx.vShieldedOutput.size() > 0) {
return MAX_PRIORITY; return MAX_PRIORITY;
} }