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

Re: [PATCH] formatting dav human-readable errors

From: Ben Reser <ben_at_reser.org>
Date: 2004-10-11 22:46:27 CEST

On Wed, Sep 29, 2004 at 04:38:42PM +0100, Philip Martin wrote:
> Ben Reser <ben@reser.org> writes:
>
> > Can you actually give a transcript of an example of when you see this
> > message.
>
> Commit an out-of-date file.
>
> $ svn ci foo
> Committed revision N.
> $ svn up -rN-1 foo
> $ echo x >> foo
> $ svn ci foo
>
> > I guess I'm failing to understand your explanation of the
> > problem you're solving. In your email you talk about the whitespace in
> > the XML.
>
> 1. mod_dav_svn generates an error text.
> 2. mod_dav adds whitespace when writing the XML.
> 3. The client parses the XML and extracts the error text complete with
> whitespace.
> 4. The client creates an svn_error_t and copies the error text complete
> with whitespace.
> 5. The svn_error_t gets returned back up the call stack in the client.
>
> I propose to remove the whitespace at step 4.

I don't see a problem with this. If someone is parsing our output they
probably are ignoring whitespace already or can easily start ignoring
it.

-- 
Ben Reser <ben@reser.org>
http://ben.reser.org
"Conscience is the inner voice which warns us somebody may be looking."
- H.L. Mencken
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Oct 11 22:46:59 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.