Re: [PATCH] Round-down years in "years+months" relative date view
On Fri, Aug 28, 2009 at 21:15, Jeff King<peff@peff.net> wrote:
Show 11 quoted lines
> On Fri, Aug 28, 2009 at 09:03:19PM +0200, Alex Riesen wrote:
>
>> +unsigned long approxidate(const char *date)
>> +{
>> + struct timeval tv;
>> + gettimeofday(&tv, NULL);
>> + return approxidate_relative(date, &tv);
>> +}
>
> This now always calls gettimeofday, whereas the original approxidate
> only did if parse_date failed.> On the other hand, refactoring the relative date code into its own
> function is probably a good thing in the long run.