[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r11172 - trunk/subversion/svnlook

From: Peter N. Lundblad <peter_at_famlundblad.se>
Date: 2004-09-29 23:08:04 CEST

On Wed, 29 Sep 2004, Philip Martin wrote:

> "Peter N. Lundblad" <peter@famlundblad.se> writes:
>
> >> Nearly, but not quite! This now fails to do EOL conversion on the
> >> printed log message so multi-line log messages on non-LF systems won't
> >> be correct, also since svn_subst_detranslate_string does do the EOL
> >> conversion the lengths won't match.
> >>
> > Did you test it? svn_cmdline_printf does LF -> CRLF on platforms that need
> > it through the C stdio streams.
>
> I didn't test it, I just noticed that you removed an EOL conversion.
> My apologies if it does work.
>
I haven't tested it on Windows either... I think it was discussed before
and the conclusion was that it is translated.

> > How do you think all our \ns in the code work?
>
> I've always assumed they are translated at compile time.
No, they are not. '\n' must be a single character if I understand
correctly.

Regards,
//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Sep 29 22:54:25 2004

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.