From: Ramkumar Ramachandra <artagnon@gmail.com>
Date: Thu, 24 Jun 2010 18:32:51 GMT
Subject: Re: [PATCH 1/9] Export parse_date_basic() to convert a date string to timestamp
Message-ID: <AANLkTikkR3KaMPcltwJslAxna25bmFixIKLsGzf55_ZW@mail.gmail.com>
In-Reply-To: <20100624105146.GA12376@burratino>

Hi Jonathan,

This specific patch is clear. I have some issues with some of the
other parts in the series.

Jonathan Nieder wrote:
> approxidate() is not appropriate for reading machine-written dates
> because it guesses instead of erroring out on malformed dates.
> parse_date() is less convenient since it returns its output as a
> string.  So export the underlying function that writes a timestamp.

Right. I couldn't justify exposing it in the series that's in master now.

> While at it, change the return value to match the usual convention:
> return 0 for success and -1 for failure.

Since I'm to blame for this change,
Acked-by: Ramkumar Ramachandra <artagnon@gmail.com>

-- Ram