From e9150f85c1a0aaac61ac94af06b7e291114b9da9 Mon Sep 17 00:00:00 2001 From: withbest Date: Mon, 29 Jul 2024 00:58:22 +0800 Subject: [PATCH] chore: fix some comments Signed-off-by: withbest --- contracts/Depot.sol | 2 +- contracts/Issuer.sol | 2 +- contracts/ProxyPerpsV2.sol | 2 +- contracts/RewardEscrowV2Frozen/IRewardEscrowV2Frozen.sol | 2 +- contracts/frozen/FuturesMarketBase.sol | 2 +- contracts/interfaces/IRewardEscrowV2.sol | 2 +- examples/browser.html | 2 +- publish/src/commands/deploy/configure-system-settings.js | 2 +- test/etherscan/spec.js | 2 +- 9 files changed, 9 insertions(+), 9 deletions(-) diff --git a/contracts/Depot.sol b/contracts/Depot.sol index c3bb6eb237..6f71652649 100644 --- a/contracts/Depot.sol +++ b/contracts/Depot.sol @@ -24,7 +24,7 @@ contract Depot is Owned, Pausable, ReentrancyGuard, MixinResolver, IDepot { /* ========== STATE VARIABLES ========== */ - // Address where the ether and Synths raised for selling SNX is transfered to + // Address where the ether and Synths raised for selling SNX is transferred to // Any ether raised for selling Synths gets sent back to whoever deposited the Synths, // and doesn't have anything to do with this address. address payable public fundsWallet; diff --git a/contracts/Issuer.sol b/contracts/Issuer.sol index 597db34ab8..50df632ef8 100644 --- a/contracts/Issuer.sol +++ b/contracts/Issuer.sol @@ -846,7 +846,7 @@ contract Issuer is Owned, MixinSystemSettings, IIssuer { uint rewardsSum ) internal view returns (uint totalRedeemed, uint escrowToLiquidate) { // The balanceOf here can be considered "transferable" since it's not escrowed, - // and it is the only SNX that can potentially be transfered if unstaked. + // and it is the only SNX that can potentially be transferred if unstaked. uint transferable = _snxBalanceOf(account); if (redeemTarget.add(rewardsSum) <= transferable) { // transferable is enough diff --git a/contracts/ProxyPerpsV2.sol b/contracts/ProxyPerpsV2.sol index cf13ed264f..b41533e49d 100644 --- a/contracts/ProxyPerpsV2.sol +++ b/contracts/ProxyPerpsV2.sol @@ -55,7 +55,7 @@ contract ProxyPerpsV2 is Owned { // Accepting a for loop since implementations for a market is going to be a very limited number (initially only 2) for (uint idx = 0; idx < _routedTargets.length; idx++) { if (_routedTargets[idx] == implementation) { - // remove it by bringing the last one to that position and poping the latest item (if it's the latest one will do an unecessary write) + // remove it by bringing the last one to that position and poping the latest item (if it's the latest one will do an unnecessary write) _routedTargets[idx] = _routedTargets[_routedTargets.length - 1]; _routedTargets.pop(); break; diff --git a/contracts/RewardEscrowV2Frozen/IRewardEscrowV2Frozen.sol b/contracts/RewardEscrowV2Frozen/IRewardEscrowV2Frozen.sol index 74fccdfb2a..75ab8867b8 100644 --- a/contracts/RewardEscrowV2Frozen/IRewardEscrowV2Frozen.sol +++ b/contracts/RewardEscrowV2Frozen/IRewardEscrowV2Frozen.sol @@ -87,7 +87,7 @@ interface IRewardEscrowV2Frozen { VestingEntries.VestingEntry[] calldata vestingEntries ) external; - // Return amount of SNX transfered to SynthetixBridgeToOptimism deposit contract + // Return amount of SNX transferred to SynthetixBridgeToOptimism deposit contract function burnForMigration(address account, uint256[] calldata entryIDs) external returns (uint256 escrowedAccountBalance, VestingEntries.VestingEntry[] memory vestingEntries); diff --git a/contracts/frozen/FuturesMarketBase.sol b/contracts/frozen/FuturesMarketBase.sol index 506ca21b72..697f54efdb 100644 --- a/contracts/frozen/FuturesMarketBase.sol +++ b/contracts/frozen/FuturesMarketBase.sol @@ -458,7 +458,7 @@ contract FuturesMarketBase is MixinFuturesMarketSettings, IFuturesMarketBaseType /// Uses the exchanger to get the dynamic fee (SIP-184) for trading from sUSD to baseAsset /// this assumes dynamic fee is symmetric in direction of trade. /// @dev this is a pretty expensive action in terms of execution gas as it queries a lot - /// of past rates from oracle. Shoudn't be much of an issue on a rollup though. + /// of past rates from oracle. Shouldn't be much of an issue on a rollup though. function _dynamicFeeRate() internal view returns (uint feeRate, bool tooVolatile) { return _exchanger().dynamicFeeRateForExchange(sUSD, baseAsset); } diff --git a/contracts/interfaces/IRewardEscrowV2.sol b/contracts/interfaces/IRewardEscrowV2.sol index b1d547deed..5c15246658 100644 --- a/contracts/interfaces/IRewardEscrowV2.sol +++ b/contracts/interfaces/IRewardEscrowV2.sol @@ -117,7 +117,7 @@ interface IRewardEscrowV2 { VestingEntries.VestingEntry[] calldata vestingEntries ) external; - // Return amount of SNX transfered to SynthetixBridgeToOptimism deposit contract + // Return amount of SNX transferred to SynthetixBridgeToOptimism deposit contract function burnForMigration(address account, uint256[] calldata entryIDs) external returns (uint256 escrowedAccountBalance, VestingEntries.VestingEntry[] memory vestingEntries); diff --git a/examples/browser.html b/examples/browser.html index faa9d02184..3aab9cb7aa 100644 --- a/examples/browser.html +++ b/examples/browser.html @@ -1,7 +1,7 @@ - Example for browser envionments + Example for browser environments diff --git a/publish/src/commands/deploy/configure-system-settings.js b/publish/src/commands/deploy/configure-system-settings.js index 688e3d539e..175d91b8f4 100644 --- a/publish/src/commands/deploy/configure-system-settings.js +++ b/publish/src/commands/deploy/configure-system-settings.js @@ -401,7 +401,7 @@ module.exports = async ({ }); const aggregatorWarningFlags = (await getDeployParameter('AGGREGATOR_WARNING_FLAGS'))[network]; - // If deploying to OVM avoid ivoking setAggregatorWarningFlags for now. + // If deploying to OVM avoid invoking setAggregatorWarningFlags for now. if (aggregatorWarningFlags && !useOvm) { await runStep({ contract: 'SystemSettings', diff --git a/test/etherscan/spec.js b/test/etherscan/spec.js index 98a62b0025..c8f37117fd 100644 --- a/test/etherscan/spec.js +++ b/test/etherscan/spec.js @@ -61,7 +61,7 @@ describe(`Etherscan on ${network}`, () => { const removeSignaturesAndVariableNames = entry => { delete entry.signature; // Some contracts, such as ProxyERC20 were deployed with different function - // input names than currently in the code, so reomve these from the check + // input names than currently in the code, so remove these from the check // specifically balanceOf(address owner) was changed to balanceOf(address account) (entry.inputs || []).forEach(input => { delete input.name;