vkonst / ctd-token

Smart-contracts for Cointed Token and Token Sale
11 stars 9 forks source link

Constructor sets owner twice #4

Closed codingupastorm closed 6 years ago

codingupastorm commented 6 years ago

Not a massive bug, just going to save you a little gas when you deploy 😉

You have owner = msg.sender; in the constructor for CtdToken, but as CtdToken inherits from Ownable it is already being set.

vkonst commented 6 years ago

Considered as code beautification Code updated

I duplicated the line in the original code to make it more readable and understandable. But it may be confusing.

So the line is deleted from the CtdToken constructor.

Thanks for your input.