Tobias Ringstrom <tobias@ringstrom.mine.nu> writes:
> Masaru Tsuchiyama wrote:
> >>>>Put simply: 'svn log' under Windows ends most lines with
> >>>>CR-CR-LF.
>
> I did some investigation, and the reason is that log_message_receiver
> insubversion/clients/cmdline/log-cmd.c uses printf to print a string
> that already has been translated by svn_subst_detranslate_string to
> use CR-NL. When printf prints that string, the NL will be translated
> to CR-NL again, and you get CR-CR-NL.
>
> Actually most users of svn_subst_detranslate_string uses printf, so I
> think the right solution is to remove the EOL translation from
> it. That would also make translate/detranslate each others inverse.
>
> Unless someone complains, I'll perform that change.
I'm not convinced that that would be a correct change. Seems like it
would be more accurate to use a binary output stream instead of going
through printf() for those places that are printing multi-lined,
detranslated strings. Also, svn_subst_detranslate_string() doesn't
repair the line-endings in (and will throw an error on) a
mixed-line-ending string. Removing this behavior and using printf()
will simply mask what might be an interesting problem with the string.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Oct 24 15:56:42 2003