Closed Jannis closed 7 years ago
This PR addresses #2 and improves #3 by using a community-audited and -tested contract for guarding against reentrancy (ReentrancyGuard from OpenZeppelin) so we don't need to roll our own logic.
ReentrancyGuard
It appears that you're right that transfer() doesn't need to be guarded but let's go ahead and merge this for now. LGTM 👍
This PR addresses #2 and improves #3 by using a community-audited and -tested contract for guarding against reentrancy (
ReentrancyGuard
from OpenZeppelin) so we don't need to roll our own logic.