Could save some gas on every transfer if storage access of mapAddress_Excluded is replaced here:
if (!mapAddress_Excluded[_from] && !mapAddress_Excluded[_to])
with:
if (_fee > 0)
then inside this if block you can also include this sentence:
totalFees += _fee;
also the initialization of totalFees is useless as it automatically gets assigned a default value of 0:
totalFees = 0;
Handle
paulius.eth
Vulnerability details
Impact
Could save some gas on every transfer if storage access of mapAddress_Excluded is replaced here: if (!mapAddress_Excluded[_from] && !mapAddress_Excluded[_to]) with: if (_fee > 0) then inside this if block you can also include this sentence: totalFees += _fee; also the initialization of totalFees is useless as it automatically gets assigned a default value of 0: totalFees = 0;