Possibly missing implementation of intended functionality

[adamkolar]

This comment:

        // Set to the index in the cooldown array (see below) that represents
        // the current cooldown duration for this Kitty. This starts at zero
        // for gen0 cats, and is initialized to floor(generation/2) for others.
        // Incremented by one for each successful breeding action, regardless
        // of whether this cat is acting as matron or sire.
        uint16 cooldownIndex;

says that cooldownIndex should be initialized to floor(generation/2) for all kitties with generation > 0. However I haven't found any code that implements this functionality, all kitties regardless of generation seem to start at 0.

[dete]

Nice catch, @adamkolar! We noticed this internally yesterday, but it's a fairly important miss so I'm going to leave this issue open for a bounty.

[kimcope]

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

Impact: Low
Likelihood: High
Points: 100

We are discounting the point number because this issue was already found and fixed internally.

Please see the final leaderboard here.