Re: [PATCH 2/2] Wire new date formats to --date=<format> parser.

From: Junio C Hamano <gitster@pobox.com>
Date: Jul 14, 2007, 06:54 UTC
Message-ID: <7vbqeflbl0.fsf@assigned-by-dhcp.cox.net>
In-Reply-To: <20070714064451.GH11809@lug-owl.de>

Jan-Benedict Glaw <jbglaw@lug-owl.de> writes:

Show 14 quoted lines

> On Fri, 2007-07-13 23:43:00 -0700, Junio C Hamano <gitster@pobox.com> wrote:
>> --- a/revision.c
>> +++ b/revision.c
>> @@ -1133,6 +1133,14 @@ int setup_revisions(int argc, const char **argv, struct rev_info *revs, const ch
>>  			if (!strncmp(arg, "--date=", 7)) {
>>  				if (!strcmp(arg + 7, "relative"))
>>  					revs->date_mode = DATE_RELATIVE;
>> +				else if (!strcmp(arg + 7, "iso8601") ||
>> +					 !strcmp(arg + 7, "iso"))
>> +					revs->date_mode = DATE_ISO8601;
>> +				else if (!strcmp(arg + 7, "rfc2822") ||
>> +					 !strcmp(arg + 7, "rfc"))
>
> Maybe also add "rfc822"?

I've thought about it, but did not bother; I suspect people would just say "rfc" anyway.