DAO.calcCurrentReward() will result in an underflow if the harvest() function is called straight after a deposit or bond.
As the mapMember_lastTime is shifted 60 seconds forward; we will be doing:
mapMember_lastTime = block.timestamp + 60
block.timestamp - mapMember_lastTime = (-)60
This will revert so i assume its not an issue; but we might want to look at adding a require in harvest or calcCurrentReward for block.timestamp > mapMember_lastTime ?
DAO.calcCurrentReward() will result in an underflow if the harvest() function is called straight after a deposit or bond.
As the mapMember_lastTime is shifted 60 seconds forward; we will be doing:
This will revert so i assume its not an issue; but we might want to look at adding a require in harvest or calcCurrentReward for block.timestamp > mapMember_lastTime ?