Gas inefficiencies in `KittyMinting`

[dapperlabs]

Gas inefficiencies in `KittyMinting`

[MacroChip]

@dete As you suspected, here are two more than can be constant https://github.com/axiomzen/cryptokitties-bounty/blob/03b51a8872d2438851d0ef77cfe29c74a079d26a/contracts/KittyOwnership.sol#L13
https://github.com/axiomzen/cryptokitties-bounty/blob/03b51a8872d2438851d0ef77cfe29c74a079d26a/contracts/KittyOwnership.sol#L14

[dete]

promoCreationLimit and gen0CreationLimit are both defined as variables

Weirdly, I just learned about constant Solidity variables just this morning, I think there some others we can probably fix, also.

createPromoKitty increments both variables

Your suggestion would result in two reads (instead of one) in createGen0Auction, but it's still a net gas savings. We'll definitely look at this!

Thanks again, @Arachnid! Love these reports!

[kimcope]

Thanks for your participation, @Arachnid! Our team has reviewed your submission, and we are pleased to reward you for your report.

Impact: Low
Likelihood: Low
Points: 50

Please see the final leaderboard here.