BAKE [Bakery] Token

BakeryToken._writeCheckpoint(address,uint32,uint256,uint256) (#1030-1046) uses a dangerous strict equality:
- nCheckpoints > 0 && checkpoints[delegatee][nCheckpoints - 1].fromBlock == blockNumber (#1038)
Don't use strict equality to determine if an account has enough Ether or tokens.

Additional information: link

Pragma version>=0.6.6 (#3) allows old versions
solc-0.6.6 is not recommended for deployment
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

Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#511-537):
- (success,returndata) = target.call{value: weiValue}(data) (#520)
Low level call in Address.sendValue(address,uint256) (#432-438):
- (success) = recipient.call{value: amount}() (#436)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence

Additional information: link

Parameter BakeryToken.mintTo(address,uint256)._to (#852) is not in mixedCase
Parameter BakeryToken.mintTo(address,uint256)._amount (#852) is not in mixedCase
Variable BakeryToken._delegates (#864) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Redundant expression "this (#25)" inContext (#15-28)
Remove redundant statements if they congest code but offer no value.

Additional information: link

SafeMath.mul(uint256,uint256) (#272-284) is never used and should be removed
Address.sendValue(address,uint256) (#432-438) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#487-493) is never used and should be removed
SafeMath.sqrt(uint256) (#368-379) is never used and should be removed
BEP20._burnFrom(address,uint256) (#840-847) is never used and should be removed
BEP20._burn(address,uint256) (#801-807) is never used and should be removed
SafeMath.min(uint256,uint256) (#363-365) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#354-361) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#501-509) is never used and should be removed
SafeMath.div(uint256,uint256,string) (#314-324) is never used and should be removed
Context._msgData() (#24-27) is never used and should be removed
SafeMath.mod(uint256,uint256) (#338-340) is never used and should be removed
SafeMath.div(uint256,uint256) (#298-300) is never used and should be removed
Address._functionCallWithValue(address,bytes,uint256,string) (#511-537) is never used and should be removed
Address.isContract(address) (#403-414) is never used and should be removed
Address.functionCall(address,bytes,string) (#468-474) is never used and should be removed
Address.functionCall(address,bytes) (#458-460) is never used and should be removed
Remove unused functions.

Additional information: link

BEP20.allowance(address,address).owner (#651) shadows:
- Ownable.owner() (#59-61) (function)
BEP20.constructor(string,string).symbol (#587) shadows:
- BEP20.symbol() (#617-619) (function)
- IBEP20.symbol() (#115) (function)
BEP20._approve(address,address,uint256).owner (#823) shadows:
- Ownable.owner() (#59-61) (function)
BEP20.constructor(string,string).name (#587) shadows:
- BEP20.name() (#603-605) (function)
- IBEP20.name() (#120) (function)
Rename the local variables that shadow another component.

Additional information: link

BakeryToken.delegateBySig(address,uint256,uint256,uint8,bytes32,bytes32) (#922-943) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(now <= expiry,BAKE::delegateBySig: signature expired) (#941)
Avoid relying on block.timestamp.

Additional information: link

Address.isContract(address) (#403-414) uses assembly
- INLINE ASM (#410-412)
BakeryToken.getChainId() (#1053-1059) uses assembly
- INLINE ASM (#1055-1057)
Address._functionCallWithValue(address,bytes,uint256,string) (#511-537) uses assembly
- INLINE ASM (#529-532)
Do not use evm assembly.

Additional information: link

transfer(address,uint256) should be declared external:
- BEP20.transfer(address,uint256) (#643-646)
symbol() should be declared external:
- BEP20.symbol() (#617-619)
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#78-81)
mintTo(address,uint256) should be declared external:
- BakeryToken.mintTo(address,uint256) (#852-855)
approve(address,uint256) should be declared external:
- BEP20.approve(address,uint256) (#675-678)
transferFrom(address,address,uint256) should be declared external:
- BEP20.transferFrom(address,address,uint256) (#692-704)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#87-89)
mint(uint256) should be declared external:
- BEP20.mint(uint256) (#663-666)
allowance(address,address) should be declared external:
- BEP20.allowance(address,address) (#651-653)
totalSupply() should be declared external:
- BEP20.totalSupply() (#624-626)
decimals() should be declared external:
- BEP20.decimals() (#610-612)
increaseAllowance(address,uint256) should be declared external:
- BEP20.increaseAllowance(address,uint256) (#718-721)
decreaseAllowance(address,uint256) should be declared external:
- BEP20.decreaseAllowance(address,uint256) (#737-744)
Use the external attribute for functions never called from the contract.

Additional information: link

Unable to find audit link on the website

Unable to find whitepaper link on the website