LYT [LooksYummy] Token

Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)

Different versions of Solidity is used:
- Version used: ['>=0.8.0', '^0.8.0']
- >=0.8.0 (#6)
- ^0.8.0 (#8)
- ^0.8.0 (#225)
- ^0.8.0 (#301)
- ^0.8.0 (#325)
- ^0.8.0 (#348)
- ^0.8.0 (#414)
Use one Solidity version.

Additional information: link

Context._msgData() (#342-345) is never used and should be removed
ERC20._burn(address,uint256) (#662-673) is never used and should be removed
SafeMath.add(uint256,uint256) (#97-99) is never used and should be removed
SafeMath.div(uint256,uint256) (#139-141) is never used and should be removed
SafeMath.div(uint256,uint256,string) (#195-200) is never used and should be removed
SafeMath.mod(uint256,uint256) (#155-157) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#217-222) is never used and should be removed
SafeMath.mul(uint256,uint256) (#125-127) is never used and should be removed
SafeMath.sub(uint256,uint256) (#111-113) is never used and should be removed
SafeMath.sub(uint256,uint256,string) (#172-177) is never used and should be removed
SafeMath.tryAdd(uint256,uint256) (#26-32) is never used and should be removed
SafeMath.tryDiv(uint256,uint256) (#68-73) is never used and should be removed
SafeMath.tryMod(uint256,uint256) (#80-85) is never used and should be removed
SafeMath.tryMul(uint256,uint256) (#51-61) is never used and should be removed
SafeMath.trySub(uint256,uint256) (#39-44) is never used and should be removed
Remove unused functions.

Additional information: link

Pragma version>=0.8.0 (#6) allows old versions
Pragma version^0.8.0 (#8) allows old versions
Pragma version^0.8.0 (#225) allows old versions
Pragma version^0.8.0 (#301) allows old versions
Pragma version^0.8.0 (#325) allows old versions
Pragma version^0.8.0 (#348) allows old versions
Pragma version^0.8.0 (#414) allows old versions
Deploy with any of the following Solidity versions: 0.5.16 - 0.5.17, 0.6.11 - 0.6.12, 0.7.5 - 0.7.6 Use a simple pragma version that allows any of these versions. Consider using the latest version of Solidity for testing.

Additional information: link

Variable LooksYummyToken.TOTALSUPPLY (#715) is not in mixedCase
Variable LooksYummyToken.Owner1 (#716) is not in mixedCase
Variable LooksYummyToken.Owner2 (#717) is not in mixedCase
Variable LooksYummyToken.Company (#718) is not in mixedCase
Variable LooksYummyToken.Appserver (#719) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Redundant expression "this (#343)" inContext (#337-346)
Remove redundant statements if they congest code but offer no value.

Additional information: link

LooksYummyToken.constructor() (#721-728) uses literals with too many digits:
- _mint(Owner1,15000000 * 1000000) (#723)
LooksYummyToken.constructor() (#721-728) uses literals with too many digits:
- _mint(Owner2,15000000 * 1000000) (#724)
LooksYummyToken.constructor() (#721-728) uses literals with too many digits:
- _mint(Company,70000000 * 1000000) (#725)
LooksYummyToken.constructor() (#721-728) uses literals with too many digits:
- _mint(Appserver,25000000 * 1000000) (#726)
LooksYummyToken.slitherConstructorVariables() (#713-734) uses literals with too many digits:
- TOTALSUPPLY = 125000000 * 1000000 (#715)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

LooksYummyToken.Appserver (#719) should be constant
LooksYummyToken.Company (#718) should be constant
LooksYummyToken.Owner1 (#716) should be constant
LooksYummyToken.Owner2 (#717) should be constant
LooksYummyToken.TOTALSUPPLY (#715) should be constant
Add the constant attributes to state variables that never change.

Additional information: link

renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#398-401)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#407-411)
name() should be declared external:
- ERC20.name() (#467-469)
symbol() should be declared external:
- ERC20.symbol() (#475-477)
decimals() should be declared external:
- ERC20.decimals() (#492-494)
- LooksYummyToken.decimals() (#730-732)
totalSupply() should be declared external:
- ERC20.totalSupply() (#499-501)
balanceOf(address) should be declared external:
- ERC20.balanceOf(address) (#506-508)
transfer(address,uint256) should be declared external:
- ERC20.transfer(address,uint256) (#518-521)
allowance(address,address) should be declared external:
- ERC20.allowance(address,address) (#526-528)
approve(address,uint256) should be declared external:
- ERC20.approve(address,uint256) (#537-540)
transferFrom(address,address,uint256) should be declared external:
- ERC20.transferFrom(address,address,uint256) (#555-563)
increaseAllowance(address,uint256) should be declared external:
- ERC20.increaseAllowance(address,uint256) (#577-580)
decreaseAllowance(address,uint256) should be declared external:
- ERC20.decreaseAllowance(address,uint256) (#596-602)
Use the external attribute for functions never called from the contract.

Additional information: link