Good point, I'll fix that.


On Sun, Nov 17, 2013 at 10:24 PM, Jag Talon <notifications@github.com>wrote:

> In lib/DDG/Goodie/LeapYear.pm:
>
> > +        $year ++;
> > +        while($numdone < $2) {
> > +            while(!isleap($year)) {
> > +                $year++;
> > +            }
> > +            $years[$numdone++] = $year++;
> > +        }
> > +        my @pretty_years = join(', ', @years);
> > +        return "The $1 $2 leap years will be @pretty_years";
> > +    } elsif ($_ =~ /(next|future|upcoming)/) {
> > +        $year ++;
> > +        while(!isleap($year)) {
> > +            $year ++;
> > +        }
> > +        return "$year will be the $1 leap year";
> > +    } elsif ($_ =~ /(last|previous|most recent)/) {
>
> If this year was a leap year, and I typed in "most recent leap year", I
> think it should display this year instead of the leap year previous to
> that. I changed $year to 2012, and I expected the query "most recent leap
> year" to return "2012 is the most recent leap year" or "This year is a leap
> year."
>
> It might just be me, though. What do you think?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/duckduckgo/zeroclickinfo-goodies/pull/236/files#r7714404>
> .
>