Re: [PATCH] formatting dav human-readable errors

[Ben Reser <be...@reser.org>]

On Wed, Sep 29, 2004 at 04:38:42PM +0100, Philip Martin wrote:
> Ben Reser <be...@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 <be...@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