Skip to content

Commit

Permalink
Add missing cases in combineCandidates
Browse files Browse the repository at this point in the history
  • Loading branch information
sisuresh committed Sep 20, 2023
1 parent f390818 commit 22088c1
Showing 1 changed file with 18 additions and 0 deletions.
18 changes: 18 additions & 0 deletions src/herder/HerderSCPDriver.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -673,6 +673,24 @@ HerderSCPDriver::combineCandidates(uint64_t slotIndex,
clUpgrade.newBaseReserve() = std::max(
clUpgrade.newBaseReserve(), lupgrade.newBaseReserve());
break;
case LEDGER_UPGRADE_FLAGS:
clUpgrade.newFlags() =
std::max(clUpgrade.newFlags(), lupgrade.newFlags());
break;
case LEDGER_UPGRADE_CONFIG:
if (clUpgrade.newConfig().contractID <
lupgrade.newConfig().contractID)
{
clUpgrade.newConfig() = lupgrade.newConfig();
}
else if (clUpgrade.newConfig().contractID ==
lupgrade.newConfig().contractID &&
clUpgrade.newConfig().contentHash <
lupgrade.newConfig().contentHash)
{
clUpgrade.newConfig() = lupgrade.newConfig();
}
break;
case LEDGER_UPGRADE_MAX_SOROBAN_TX_SET_SIZE:
clUpgrade.newMaxSorobanTxSetSize() =
std::max(clUpgrade.newMaxSorobanTxSetSize(),
Expand Down

5 comments on commit 22088c1

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from dmkozh
at sisuresh@22088c1

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging sisuresh/stellar-core/exp-db-fix = 22088c1 into auto

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sisuresh/stellar-core/exp-db-fix = 22088c1 merged ok, testing candidate = 6e71a1e

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = 6e71a1e

Please sign in to comment.