From e222ff58680d884a46b5bbf581db2dbf621e7233 Mon Sep 17 00:00:00 2001 From: Joe Loser Date: Fri, 29 Jun 2018 07:38:20 -0400 Subject: [PATCH] Rename data members of ConsensusParms: Based on a TODO comment in DisputedTX.h, it seems at one point the data members of ConsensusParms were macros. Now that they are not, we should spell them like other data members (without all uppercase). --- src/ripple/consensus/DisputedTx.h | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/ripple/consensus/DisputedTx.h b/src/ripple/consensus/DisputedTx.h index 93612e5818..adfa218dfa 100644 --- a/src/ripple/consensus/DisputedTx.h +++ b/src/ripple/consensus/DisputedTx.h @@ -213,10 +213,6 @@ DisputedTx::updateVote( // This is basically the percentage of nodes voting 'yes' (including us) weight = (yays_ * 100 + (ourVote_ ? 100 : 0)) / (nays_ + yays_ + 1); - // VFALCO TODO Rename these macros and turn them into language - // constructs. consolidate them into a class that collects - // all these related values. - // // To prevent avalanche stalls, we increase the needed weight slightly // over time. if (percentTime < p.avMID_CONSENSUS_TIME)