bokkypoobah / FixedSupplyTokenFactory

Fixed Supply Token 👊 Contract + Factory - A Token Contract Vending Machine

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Cosmetics including `address payable`

bokkypoobah opened this issue · comments

@pertsev provided some cosmetic suggestions.

I will not be marking owner and newOwner as payable, as suggested, as the payable prevents an address variable from accidentally being used in a .transfer(...).

I will be leaving the address(uint160(owner)).transfer(...) casting in place, as this explicitly marks the statement where the transfer is made. For me, at least