STINE [Shibatine] Token

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

Reentrancy in Shibatine._transfer(address,address,uint256) (#664-756):
External calls:
- swapBack() (#720)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0xdead),block.timestamp) (#813-820)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#802-808)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#794)
External calls sending eth:
- swapBack() (#720)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0xdead),block.timestamp) (#813-820)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#794)
State variables written after the call(s):
- _tokenTransfer(from,to,amount) (#752)
- _liquidityTokensToSwap += tLiquidity * _buyLiquidityFee / _liquidityFee (#1020)
- _liquidityTokensToSwap += tLiquidity * _sellLiquidityFee / _liquidityFee (#1023)
- _tokenTransfer(from,to,amount) (#752)
- _marketingTokensToSwap += tLiquidity * _buyMarketingFee / _liquidityFee (#1021)
- _marketingTokensToSwap += tLiquidity * _sellMarketingFee / _liquidityFee (#1024)
- _tokenTransfer(from,to,amount) (#752)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#873)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#853)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#854)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#895)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#916)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#875)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#896)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity) (#1028)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#918)
- _tokenTransfer(from,to,amount) (#752)
- _rTotal = _rTotal.sub(rFee) (#925)
- _tokenTransfer(from,to,amount) (#752)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#894)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#915)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#874)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#917)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity) (#1030)
Apply the check-effects-interactions pattern.

Additional information: link

Ownable._lockTime (#209) is never initialized. It is used in:
- Ownable.getUnlockTime() (#245-247)
Initialize all the variables. If a variable is meant to be initialized to zero, explicitly set it to zero to improve code readability.

Additional information: link

Reentrancy in Shibatine.constructor() (#405-433):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#412)
State variables written after the call(s):
- transferOwnership(newOwner) (#432)
- _owner = newOwner (#242)
Reentrancy in Shibatine.swapBack() (#758-795):
External calls:
- swapTokensForETH(amountToSwapForETH) (#777)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#802-808)
State variables written after the call(s):
- _liquidityTokensToSwap = 0 (#785)
- _marketingTokensToSwap = 0 (#786)
Apply the check-effects-interactions pattern.

Additional information: link

Shibatine._transfer(address,address,uint256) (#664-756) uses tx.origin for authorization: require(bool,string)(_holderLastTransferTimestamp[tx.origin] < block.number - 1 && _holderLastTransferTimestamp[to] < block.number - 1,_transfer:: Transfer Delay enabled. Only one purchase per block allowed.) (#690)
Do not use tx.origin for authorization.

Additional information: link

Shibatine.addLiquidity(uint256,uint256) (#811-821) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0xdead),block.timestamp) (#813-820)
Ensure that all the return values of the function calls are used.

Additional information: link

Shibatine.allowance(address,address).owner (#465) shadows:
- Ownable.owner() (#222-224) (function)
Shibatine._approve(address,address,uint256).owner (#653) shadows:
- Ownable.owner() (#222-224) (function)
Rename the local variables that shadow another component.

Additional information: link

Shibatine.updateMinimumTokensBeforeSwap(uint256) (#577-581) should emit an event for:
- minimumTokensBeforeSwap = newAmount (#580)
Shibatine.updateMaxAmount(uint256) (#583-586) should emit an event for:
- maxTransactionAmount = newNum * (10 ** 18) (#585)
Emit an event for critical parameter changes.

Additional information: link

Reentrancy in Shibatine._transfer(address,address,uint256) (#664-756):
External calls:
- swapBack() (#720)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0xdead),block.timestamp) (#813-820)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#802-808)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#794)
External calls sending eth:
- swapBack() (#720)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0xdead),block.timestamp) (#813-820)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#794)
State variables written after the call(s):
- removeAllFee() (#723)
- _liquidityFee = 0 (#1052)
- _liquidityFee = 9300 (#731)
- _liquidityFee = _buyLiquidityFee + _buyMarketingFee (#737)
- _liquidityFee = _sellLiquidityFee + _sellMarketingFee (#745)
- restoreAllFee() (#754)
- _liquidityFee = _previousLiquidityFee (#1057)
- removeAllFee() (#723)
- _previousLiquidityFee = _liquidityFee (#1049)
- removeAllFee() (#723)
- _previousTaxFee = _taxFee (#1048)
- _tokenTransfer(from,to,amount) (#752)
- _tFeeTotal = _tFeeTotal.add(tFee) (#926)
- removeAllFee() (#723)
- _taxFee = 0 (#1051)
- _taxFee = 600 (#730)
- _taxFee = _buyTaxFee (#736)
- _taxFee = _sellTaxFee (#744)
- restoreAllFee() (#754)
- _taxFee = _previousTaxFee (#1056)
- buyOrSellSwitch = TRANSFER (#725)
- buyOrSellSwitch = SELL (#732)
- buyOrSellSwitch = BUY (#739)
- buyOrSellSwitch = SELL (#747)
Reentrancy in Shibatine.constructor() (#405-433):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#412)
State variables written after the call(s):
- _setAutomatedMarketMakerPair(address(uniswapV2Pair),true) (#414)
- _excluded.push(account) (#636)
- _excluded[i] = _excluded[_excluded.length - 1] (#643)
- _excluded.pop() (#646)
- excludeFromReward(address(this)) (#429)
- _excluded.push(account) (#636)
- _setAutomatedMarketMakerPair(address(uniswapV2Pair),true) (#414)
- _isExcluded[account] = true (#635)
- _isExcluded[account] = false (#645)
- excludeFromReward(address(this)) (#429)
- _isExcluded[account] = true (#635)
- excludeFromFee(newOwner) (#421)
- _isExcludedFromFee[account] = true (#1065)
- excludeFromFee(address(this)) (#422)
- _isExcludedFromFee[account] = true (#1065)
- excludeFromFee(address(0xdead)) (#423)
- _isExcludedFromFee[account] = true (#1065)
- excludeFromMaxTransaction(address(uniswapV2Pair),true) (#413)
- _isExcludedMaxTransactionAmount[updAds] = isEx (#561)
- _setAutomatedMarketMakerPair(address(uniswapV2Pair),true) (#414)
- _isExcludedMaxTransactionAmount[pair] = value (#596)
- excludeFromMaxTransaction(newOwner,true) (#425)
- _isExcludedMaxTransactionAmount[updAds] = isEx (#561)
- excludeFromMaxTransaction(address(this),true) (#426)
- _isExcludedMaxTransactionAmount[updAds] = isEx (#561)
- excludeFromMaxTransaction(address(0xdead),true) (#427)
- _isExcludedMaxTransactionAmount[updAds] = isEx (#561)
- _setAutomatedMarketMakerPair(address(uniswapV2Pair),true) (#414)
- _tOwned[account] = tokenFromReflection(_rOwned[account]) (#633)
- _tOwned[account] = 0 (#644)
- excludeFromReward(address(this)) (#429)
- _tOwned[account] = tokenFromReflection(_rOwned[account]) (#633)
- _setAutomatedMarketMakerPair(address(uniswapV2Pair),true) (#414)
- automatedMarketMakerPairs[pair] = value (#595)
- marketingAddress = address(0x308939fF2D74F6002E16DB303ffDf23F74Add957) (#419)
- maxTransactionAmount = _tTotal * 5 / 1000 (#416)
- minimumTokensBeforeSwap = _tTotal * 5 / 10000 (#417)
Reentrancy in Shibatine.swapBack() (#758-795):
External calls:
- swapTokensForETH(amountToSwapForETH) (#777)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#802-808)
- addLiquidity(tokensForLiquidity,ethForLiquidity) (#789)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0xdead),block.timestamp) (#813-820)
External calls sending eth:
- addLiquidity(tokensForLiquidity,ethForLiquidity) (#789)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0xdead),block.timestamp) (#813-820)
State variables written after the call(s):
- addLiquidity(tokensForLiquidity,ethForLiquidity) (#789)
- _allowances[owner][spender] = amount (#660)
Reentrancy in Shibatine.transferFrom(address,address,uint256) (#483-498):
External calls:
- _transfer(sender,recipient,amount) (#488)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0xdead),block.timestamp) (#813-820)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#802-808)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#794)
External calls sending eth:
- _transfer(sender,recipient,amount) (#488)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0xdead),block.timestamp) (#813-820)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#794)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#489-496)
- _allowances[owner][spender] = amount (#660)
Apply the check-effects-interactions pattern.

Additional information: link

Reentrancy in Shibatine._transfer(address,address,uint256) (#664-756):
External calls:
- swapBack() (#720)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0xdead),block.timestamp) (#813-820)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#802-808)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#794)
External calls sending eth:
- swapBack() (#720)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0xdead),block.timestamp) (#813-820)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#794)
Event emitted after the call(s):
- Transfer(sender,recipient,tTransferAmount) (#857)
- _tokenTransfer(from,to,amount) (#752)
- Transfer(sender,recipient,tTransferAmount) (#899)
- _tokenTransfer(from,to,amount) (#752)
- Transfer(sender,recipient,tTransferAmount) (#878)
- _tokenTransfer(from,to,amount) (#752)
- Transfer(sender,recipient,tTransferAmount) (#921)
- _tokenTransfer(from,to,amount) (#752)
Reentrancy in Shibatine.constructor() (#405-433):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#412)
Event emitted after the call(s):
- ExcludeFromFee(account) (#1066)
- excludeFromFee(address(0xdead)) (#423)
- ExcludeFromFee(account) (#1066)
- excludeFromFee(address(this)) (#422)
- ExcludeFromFee(account) (#1066)
- excludeFromFee(newOwner) (#421)
- OwnershipTransferred(_owner,newOwner) (#241)
- transferOwnership(newOwner) (#432)
- Transfer(address(0),newOwner,_tTotal) (#431)
Reentrancy in Shibatine.swapBack() (#758-795):
External calls:
- swapTokensForETH(amountToSwapForETH) (#777)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#802-808)
- addLiquidity(tokensForLiquidity,ethForLiquidity) (#789)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0xdead),block.timestamp) (#813-820)
External calls sending eth:
- addLiquidity(tokensForLiquidity,ethForLiquidity) (#789)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0xdead),block.timestamp) (#813-820)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#661)
- addLiquidity(tokensForLiquidity,ethForLiquidity) (#789)
- SwapAndLiquify(amountToSwapForETH,ethForLiquidity,tokensForLiquidity) (#790)
Reentrancy in Shibatine.transferForeignToken(address,address) (#1110-1120):
External calls:
- _sent = IERC20(_token).transfer(_to,_contractBalance) (#1118)
Event emitted after the call(s):
- TransferForeignToken(_token,_contractBalance) (#1119)
Reentrancy in Shibatine.transferFrom(address,address,uint256) (#483-498):
External calls:
- _transfer(sender,recipient,amount) (#488)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0xdead),block.timestamp) (#813-820)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#802-808)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#794)
External calls sending eth:
- _transfer(sender,recipient,amount) (#488)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0xdead),block.timestamp) (#813-820)
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#794)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#661)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#489-496)
Apply the check-effects-interactions pattern.

Additional information: link

Address.isContract(address) (#111-122) uses assembly
- INLINE ASM (#118-120)
Address._functionCallWithValue(address,bytes,uint256,string) (#180-203) uses assembly
- INLINE ASM (#195-198)
Do not use evm assembly.

Additional information: link

Shibatine.includeInReward(address) (#639-650) has costly operations inside a loop:
- _excluded.pop() (#646)
Shibatine._takeLiquidity(uint256) (#1018-1031) has costly operations inside a loop:
- _liquidityTokensToSwap += tLiquidity * _buyLiquidityFee / _liquidityFee (#1020)
Shibatine._takeLiquidity(uint256) (#1018-1031) has costly operations inside a loop:
- _marketingTokensToSwap += tLiquidity * _buyMarketingFee / _liquidityFee (#1021)
Shibatine._takeLiquidity(uint256) (#1018-1031) has costly operations inside a loop:
- _liquidityTokensToSwap += tLiquidity * _sellLiquidityFee / _liquidityFee (#1023)
Shibatine._takeLiquidity(uint256) (#1018-1031) has costly operations inside a loop:
- _marketingTokensToSwap += tLiquidity * _sellMarketingFee / _liquidityFee (#1024)
Shibatine._reflectFee(uint256,uint256) (#924-927) has costly operations inside a loop:
- _rTotal = _rTotal.sub(rFee) (#925)
Shibatine._reflectFee(uint256,uint256) (#924-927) has costly operations inside a loop:
- _tFeeTotal = _tFeeTotal.add(tFee) (#926)
Use a local variable to hold the loop computation result.

Additional information: link

Address._functionCallWithValue(address,bytes,uint256,string) (#180-203) is never used and should be removed
Address.functionCall(address,bytes) (#138-143) is never used and should be removed
Address.functionCall(address,bytes,string) (#145-151) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#153-165) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#167-178) is never used and should be removed
Address.isContract(address) (#111-122) is never used and should be removed
Address.sendValue(address,uint256) (#124-136) is never used and should be removed
Context._msgData() (#10-13) is never used and should be removed
SafeMath.mod(uint256,uint256) (#96-98) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#100-107) is never used and should be removed
Remove unused functions.

Additional information: link

Shibatine._previousTaxFee (#329) is set pre-construction with a non-constant function or state variable:
- _taxFee
Shibatine._previousLiquidityFee (#332) is set pre-construction with a non-constant function or state variable:
- _liquidityFee
Remove any initialization of state variables via non-constant state variables or function calls. If variables must be set upon contract deployment, locate initialization in the constructor instead.

Additional information: link

Pragma version0.8.11 (#3) necessitates a version too recent to be trusted. Consider deploying with 0.6.12/0.7.6/0.8.7
solc-0.8.11 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.sendValue(address,uint256) (#124-136):
- (success) = recipient.call{value: amount}() (#131)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#180-203):
- (success,returndata) = target.call{value: weiValue}(data) (#188-190)
Low level call in Shibatine.swapBack() (#758-795):
- (success,None) = address(marketingAddress).call{value: address(this).balance}() (#794)
Low level call in Shibatine.withdrawStuckETH() (#1123-1126):
- (success,None) = address(msg.sender).call{value: address(this).balance}() (#1125)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence

Additional information: link

Function IUniswapV2Router02.WETH() (#264) is not in mixedCase
Parameter Shibatine.calculateTaxFee(uint256)._amount (#1033) is not in mixedCase
Parameter Shibatine.calculateLiquidityFee(uint256)._amount (#1037) is not in mixedCase
Parameter Shibatine.setMarketingAddress(address)._marketingAddress (#1096) is not in mixedCase
Parameter Shibatine.setSwapAndLiquifyEnabled(bool)._enabled (#1102) is not in mixedCase
Parameter Shibatine.transferForeignToken(address,address)._token (#1110) is not in mixedCase
Parameter Shibatine.transferForeignToken(address,address)._to (#1110) is not in mixedCase
Constant Shibatine._tTotal (#319) is not in UPPER_CASE_WITH_UNDERSCORES
Constant Shibatine._name (#323) is not in UPPER_CASE_WITH_UNDERSCORES
Constant Shibatine._symbol (#324) is not in UPPER_CASE_WITH_UNDERSCORES
Constant Shibatine._decimals (#325) is not in UPPER_CASE_WITH_UNDERSCORES
Variable Shibatine._buyTaxFee (#339) is not in mixedCase
Variable Shibatine._buyLiquidityFee (#340) is not in mixedCase
Variable Shibatine._buyMarketingFee (#341) is not in mixedCase
Variable Shibatine._sellTaxFee (#343) is not in mixedCase
Variable Shibatine._sellLiquidityFee (#344) is not in mixedCase
Variable Shibatine._sellMarketingFee (#345) is not in mixedCase
Variable Shibatine._liquidityTokensToSwap (#351) is not in mixedCase
Variable Shibatine._marketingTokensToSwap (#352) is not in mixedCase
Variable Shibatine._isExcludedMaxTransactionAmount (#355) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Redundant expression "this (#11)" inContext (#5-14)
Remove redundant statements if they congest code but offer no value.

Additional information: link

Variable Shibatine._transferBothExcluded(address,address,uint256).rTransferAmount (#909) is too similar to Shibatine._transferToExcluded(address,address,uint256).tTransferAmount (#869)
Variable Shibatine.reflectionFromToken(uint256,bool).rTransferAmount (#611) is too similar to Shibatine._transferFromExcluded(address,address,uint256).tTransferAmount (#890)
Variable Shibatine._transferStandard(address,address,uint256).rTransferAmount (#847) is too similar to Shibatine._getValues(uint256).tTransferAmount (#942)
Variable Shibatine._transferToExcluded(address,address,uint256).rTransferAmount (#867) is too similar to Shibatine._transferToExcluded(address,address,uint256).tTransferAmount (#869)
Variable Shibatine.reflectionFromToken(uint256,bool).rTransferAmount (#611) is too similar to Shibatine._transferBothExcluded(address,address,uint256).tTransferAmount (#911)
Variable Shibatine._transferStandard(address,address,uint256).rTransferAmount (#847) is too similar to Shibatine._transferStandard(address,address,uint256).tTransferAmount (#849)
Variable Shibatine.reflectionFromToken(uint256,bool).rTransferAmount (#611) is too similar to Shibatine._transferToExcluded(address,address,uint256).tTransferAmount (#869)
Variable Shibatine._transferBothExcluded(address,address,uint256).rTransferAmount (#909) is too similar to Shibatine._getValues(uint256).tTransferAmount (#942)
Variable Shibatine._transferFromExcluded(address,address,uint256).rTransferAmount (#888) is too similar to Shibatine._transferFromExcluded(address,address,uint256).tTransferAmount (#890)
Variable Shibatine._transferToExcluded(address,address,uint256).rTransferAmount (#867) is too similar to Shibatine._getValues(uint256).tTransferAmount (#942)
Variable Shibatine._transferFromExcluded(address,address,uint256).rTransferAmount (#888) is too similar to Shibatine._transferBothExcluded(address,address,uint256).tTransferAmount (#911)
Variable Shibatine.reflectionFromToken(uint256,bool).rTransferAmount (#611) is too similar to Shibatine._getTValues(uint256).tTransferAmount (#973)
Variable Shibatine._transferFromExcluded(address,address,uint256).rTransferAmount (#888) is too similar to Shibatine._transferToExcluded(address,address,uint256).tTransferAmount (#869)
Variable Shibatine.reflectionFromToken(uint256,bool).rTransferAmount (#611) is too similar to Shibatine._getValues(uint256).tTransferAmount (#942)
Variable Shibatine._transferBothExcluded(address,address,uint256).rTransferAmount (#909) is too similar to Shibatine._transferStandard(address,address,uint256).tTransferAmount (#849)
Variable Shibatine._transferToExcluded(address,address,uint256).rTransferAmount (#867) is too similar to Shibatine._transferStandard(address,address,uint256).tTransferAmount (#849)
Variable Shibatine._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#994) is too similar to Shibatine._getValues(uint256).tTransferAmount (#942)
Variable Shibatine.reflectionFromToken(uint256,bool).rTransferAmount (#611) is too similar to Shibatine._transferStandard(address,address,uint256).tTransferAmount (#849)
Variable Shibatine._getValues(uint256).rTransferAmount (#946) is too similar to Shibatine._getValues(uint256).tTransferAmount (#942)
Variable Shibatine._transferFromExcluded(address,address,uint256).rTransferAmount (#888) is too similar to Shibatine._getValues(uint256).tTransferAmount (#942)
Variable Shibatine._transferFromExcluded(address,address,uint256).rTransferAmount (#888) is too similar to Shibatine._transferStandard(address,address,uint256).tTransferAmount (#849)
Variable Shibatine._transferBothExcluded(address,address,uint256).rTransferAmount (#909) is too similar to Shibatine._transferFromExcluded(address,address,uint256).tTransferAmount (#890)
Variable Shibatine._transferToExcluded(address,address,uint256).rTransferAmount (#867) is too similar to Shibatine._transferFromExcluded(address,address,uint256).tTransferAmount (#890)
Variable Shibatine._transferBothExcluded(address,address,uint256).rTransferAmount (#909) is too similar to Shibatine._transferBothExcluded(address,address,uint256).tTransferAmount (#911)
Variable Shibatine._transferToExcluded(address,address,uint256).rTransferAmount (#867) is too similar to Shibatine._transferBothExcluded(address,address,uint256).tTransferAmount (#911)
Variable Shibatine._transferStandard(address,address,uint256).rTransferAmount (#847) is too similar to Shibatine._getTValues(uint256).tTransferAmount (#973)
Variable Shibatine._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#994) is too similar to Shibatine._transferFromExcluded(address,address,uint256).tTransferAmount (#890)
Variable Shibatine._getValues(uint256).rTransferAmount (#946) is too similar to Shibatine._transferBothExcluded(address,address,uint256).tTransferAmount (#911)
Variable Shibatine._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#994) is too similar to Shibatine._getTValues(uint256).tTransferAmount (#973)
Variable Shibatine._getValues(uint256).rTransferAmount (#946) is too similar to Shibatine._transferToExcluded(address,address,uint256).tTransferAmount (#869)
Variable Shibatine._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#994) is too similar to Shibatine._transferStandard(address,address,uint256).tTransferAmount (#849)
Variable Shibatine._getValues(uint256).rTransferAmount (#946) is too similar to Shibatine._transferFromExcluded(address,address,uint256).tTransferAmount (#890)
Variable Shibatine._transferStandard(address,address,uint256).rTransferAmount (#847) is too similar to Shibatine._transferBothExcluded(address,address,uint256).tTransferAmount (#911)
Variable Shibatine._transferFromExcluded(address,address,uint256).rTransferAmount (#888) is too similar to Shibatine._getTValues(uint256).tTransferAmount (#973)
Variable Shibatine._transferBothExcluded(address,address,uint256).rTransferAmount (#909) is too similar to Shibatine._getTValues(uint256).tTransferAmount (#973)
Variable Shibatine._transferToExcluded(address,address,uint256).rTransferAmount (#867) is too similar to Shibatine._getTValues(uint256).tTransferAmount (#973)
Variable Shibatine._transferStandard(address,address,uint256).rTransferAmount (#847) is too similar to Shibatine._transferToExcluded(address,address,uint256).tTransferAmount (#869)
Variable Shibatine._getValues(uint256).rTransferAmount (#946) is too similar to Shibatine._getTValues(uint256).tTransferAmount (#973)
Variable Shibatine._getValues(uint256).rTransferAmount (#946) is too similar to Shibatine._transferStandard(address,address,uint256).tTransferAmount (#849)
Variable Shibatine._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#994) is too similar to Shibatine._transferBothExcluded(address,address,uint256).tTransferAmount (#911)
Variable Shibatine._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#994) is too similar to Shibatine._transferToExcluded(address,address,uint256).tTransferAmount (#869)
Variable Shibatine._transferStandard(address,address,uint256).rTransferAmount (#847) is too similar to Shibatine._transferFromExcluded(address,address,uint256).tTransferAmount (#890)
Prevent variables from having similar names.

Additional information: link

Shibatine.updateMinimumTokensBeforeSwap(uint256) (#577-581) uses literals with too many digits:
- require(bool,string)(newAmount >= _tTotal * 1 / 100000,Swap amount cannot be lower than 0.001% total supply.) (#578)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

Ownable._previousOwner (#208) is never used in Shibatine (#298-1141)
Remove unused state variables.

Additional information: link

Ownable._lockTime (#209) should be constant
Ownable._previousOwner (#208) should be constant
Add the constant attributes to state variables that never change.

Additional information: link

renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#231-234)
getUnlockTime() should be declared external:
- Ownable.getUnlockTime() (#245-247)
getTime() should be declared external:
- Ownable.getTime() (#249-251)
approve(address,uint256) should be declared external:
- Shibatine.approve(address,uint256) (#474-481)
setAutomatedMarketMakerPair(address,bool) should be declared external:
- Shibatine.setAutomatedMarketMakerPair(address,bool) (#588-592)
setSwapAndLiquifyEnabled(bool) should be declared external:
- Shibatine.setSwapAndLiquifyEnabled(bool) (#1102-1105)
Use the external attribute for functions never called from the contract.

Additional information: link