Re: [PATCH] $LastChangedDate$ encoding

Peter Samuelson writes:
>
> Ivan argued for not localising the keyword expansion at all, but using
> English with ASCII. [Note: I think that is reasonable, but perhaps not
> easy to implement efficiently, as it may involve switching locales.]
>
>

I thin we should go this route. Specifically, we should drop the
human readable pdate part and use the same format as Id uses. I think
it was a mistake to use a locale-dependent date format here.

This does not solve the problem, because we still use UTF8 for the
basename and author. I think the best we can do is to stay with UTF8
until we know the encoding of the file - it will at least give
consistent results.

Changing the $Date$ expansion will break compatibility, but we will
keep the same format for the machine-parsable part as we have today.

I'd be +1 on a backport of this to 1.4.

Regards,
//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun May 7 22:16:32 2006