rename variable to be more accurate according to last commit's logic

This commit is contained in:
sneurlax 2023-10-21 16:53:45 -05:00
parent 93a0f1f6ab
commit c761b9ae65

View file

@ -461,7 +461,7 @@ class BitcoinCashWallet extends CoinServiceAPI
} }
Future<Tuple3<List<isar_models.Address>, DerivePathType, int>> _checkGaps( Future<Tuple3<List<isar_models.Address>, DerivePathType, int>> _checkGaps(
int maxNumberOfIndexesToCheck, int minNumberOfIndexesToCheck,
int maxUnusedAddressGap, int maxUnusedAddressGap,
int txCountBatchSize, int txCountBatchSize,
bip32.BIP32 root, bip32.BIP32 root,
@ -472,12 +472,13 @@ class BitcoinCashWallet extends CoinServiceAPI
int gapCounter = 0; int gapCounter = 0;
int highestIndexWithHistory = 0; int highestIndexWithHistory = 0;
// Loop until either the index limit or the highest index with history, // Scan addresses until the minimum required addresses have been scanned or
// whichever is greater, plus the gap limit, so that we if there is activity // until the highest index with activity, plus the gap limit, whichever is
// above the max index, the limit is raised to that index plus the gap limit. // higher, so that we if there is activity above the minimum index, we don't
// miss it.
for (int index = 0; for (int index = 0;
index < index <
max(maxNumberOfIndexesToCheck, max(minNumberOfIndexesToCheck,
highestIndexWithHistory + maxUnusedAddressGap) && highestIndexWithHistory + maxUnusedAddressGap) &&
gapCounter < maxUnusedAddressGap; gapCounter < maxUnusedAddressGap;
index += txCountBatchSize) { index += txCountBatchSize) {