On Tue, Apr 16, 2013 at 04:22:25PM -0700, Junio C Hamano wrote:

> +static void parse_expire_value(const char *value, unsigned long *expire)
> +{
> +     if (!strcmp(value, "never") || !strcmp(value, "false"))
> +             *expire = 0;
> +     else if (!strcmp(value, "all") || !strcmp(value, "now"))
> +             /*
> +              * We take over "now" here, which usually translates
> +              * to the current timestamp, because the user really
> +              * means everything she has done in the past, and by
> +              * definition reflogs are the record of the past,
> +              * there is nothing from the future to be kept.
> +              */
> +             *expire = ULONG_MAX;
> +     else
> +             *expire = approxidate(value);
> +}

Do we want to use approxidate_careful here to catch other junk?

-Peff
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to