feat(ctb): numOwnersAfter is a better var name

a5a723d3 · Maurelian · 2d7e3ba8 · a5a723d3
Commit a5a723d3 authored Oct 24, 2023 by Maurelian
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 3 deletions

LivenessModule.sol packages/contracts-bedrock/src/Safe/LivenessModule.sol +3 -3

No files found.
--- a/packages/contracts-bedrock/src/Safe/LivenessModule.sol
+++ b/packages/contracts-bedrock/src/Safe/LivenessModule.sol
@@ -74,11 +74,11 @@ contract LivenessModule is ISemver {

        // Calculate the new threshold
        address[] memory owners = SAFE.getOwners();
-        uint256 numOwners = owners.length - 1;
+        uint256 numOwnersAfter = owners.length - 1;
        uint256 thresholdAfter;
-        if (_isAboveMinOwners(numOwners)) {
+        if (_isAboveMinOwners(numOwnersAfter)) {
            // Call the Safe to remove the owner and update the threshold
-            thresholdAfter = get75PercentThreshold(numOwners);
+            thresholdAfter = get75PercentThreshold(numOwnersAfter);
            address prevOwner = _getPrevOwner(owner, owners);
            _removeOwner({ _prevOwner: prevOwner, _owner: owner, _threshold: thresholdAfter });
        } else {