Decentralized Cross-Chain Exchange & Farming Platform MacaronSwap uses an automated market maker (AMM) model. That means that while you can trade digital assets on the platform, there isn’t an order book where you’re matched with someone else. Instead, you trade against a liquidity pool. Those pools are filled with other users’ funds. They deposit them into the pool, receiving liquidity provider (or LP) tokens in return. They can use those tokens to reclaim their share, plus a portion of the trading fees.
MacaronToken._writeCheckpoint(address,uint32,uint256,uint256) (#1075-1093) uses a dangerous strict equality:
- nCheckpoints > 0 && checkpoints[delegatee][nCheckpoints - 1].fromBlock == blockNumber (#1085)
Don't use strict equality to determine if an account has enough Ether or tokens.
Additional information: link
MacaronToken.delegateBySig(address,uint256,uint256,uint8,bytes32,bytes32) (#941-982) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(now <= expiry,MACARON::delegateBySig: signature expired) (#980)
Avoid relying on block.timestamp.
Additional information: link
Redundant expression "this (#38)" inContext (#28-41)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Contract ownership is semi-renounced (passed to a contract)
SafeMath.mul(uint256,uint256) (#288-300) is never used and should be removed
Address.sendValue(address,uint256) (#449-455) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#504-510) is never used and should be removed
SafeMath.sqrt(uint256) (#384-395) is never used and should be removed
BEP20._burnFrom(address,uint256) (#858-865) is never used and should be removed
BEP20._burn(address,uint256) (#819-825) is never used and should be removed
SafeMath.min(uint256,uint256) (#379-381) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#370-377) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#518-526) is never used and should be removed
SafeMath.div(uint256,uint256,string) (#330-340) is never used and should be removed
Context._msgData() (#37-40) is never used and should be removed
SafeMath.mod(uint256,uint256) (#354-356) is never used and should be removed
SafeMath.div(uint256,uint256) (#314-316) is never used and should be removed
Address._functionCallWithValue(address,bytes,uint256,string) (#528-554) is never used and should be removed
Address.isContract(address) (#420-431) is never used and should be removed
Address.functionCall(address,bytes,string) (#485-491) is never used and should be removed
Address.functionCall(address,bytes) (#475-477) is never used and should be removed
Remove unused functions.
Additional information: link
Low level call in Address.sendValue(address,uint256) (#449-455):
- (success) = recipient.call{value: amount}() (#453)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#528-554):
- (success,returndata) = target.call{value: weiValue}(data) (#537)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence
Additional information: link
Variable MacaronToken._delegates (#883) is not in mixedCase
Parameter MacaronToken.mint(address,uint256)._amount (#871) is not in mixedCase
Parameter MacaronToken.mint(address,uint256)._to (#871) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
transfer(address,uint256) should be declared external:
- BEP20.transfer(address,uint256) (#661-664)
symbol() should be declared external:
- BEP20.symbol() (#635-637)
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#92-95)
approve(address,uint256) should be declared external:
- BEP20.approve(address,uint256) (#680-683)
mint(address,uint256) should be declared external:
- MacaronToken.mint(address,uint256) (#871-874)
transferFrom(address,address,uint256) should be declared external:
- BEP20.transferFrom(address,address,uint256) (#697-709)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#101-103)
mint(uint256) should be declared external:
- BEP20.mint(uint256) (#759-762)
allowance(address,address) should be declared external:
- BEP20.allowance(address,address) (#669-671)
totalSupply() should be declared external:
- BEP20.totalSupply() (#642-644)
decimals() should be declared external:
- BEP20.decimals() (#628-630)
increaseAllowance(address,uint256) should be declared external:
- BEP20.increaseAllowance(address,uint256) (#723-726)
decreaseAllowance(address,uint256) should be declared external:
- BEP20.decreaseAllowance(address,uint256) (#742-749)
Use the external attribute for functions never called from the contract.
Additional information: link
BEP20.allowance(address,address).owner (#669) shadows:
- Ownable.owner() (#73-75) (function)
BEP20.constructor(string,string).symbol (#605) shadows:
- BEP20.symbol() (#635-637) (function)
- IBEP20.symbol() (#130) (function)
BEP20._approve(address,address,uint256).owner (#841) shadows:
- Ownable.owner() (#73-75) (function)
BEP20.constructor(string,string).name (#605) shadows:
- BEP20.name() (#621-623) (function)
- IBEP20.name() (#135) (function)
Rename the local variables that shadow another component.
Additional information: link
Address.isContract(address) (#420-431) uses assembly
- INLINE ASM (#427-429)
MacaronToken.getChainId() (#1100-1104) uses assembly
- INLINE ASM (#1102)
Address._functionCallWithValue(address,bytes,uint256,string) (#528-554) uses assembly
- INLINE ASM (#546-549)
Do not use evm assembly.
Additional information: link
Token has relatively low CoinMarketCap rank
Twitter account link seems to be invalid
Unable to find Youtube account
Unable to find Discord account