Re: [PATCH] bugfix auto-props

Martin Furter wrote:
>
> On Tue, 7 Oct 2003, Julian Foad wrote:
>
>>Philip Martin wrote:
>>
>>>julianfoad@tigris.org writes:
>>>
>>>> trunk/subversion/libsvn_client/add.c
>>>
>>>You didn't fix this warning
>>
>>Ah - I didn't see them scroll by. Fixed those, though not very
>>elegantly (with a cast), and also one in
>>subversion/clients/cmdline/info-cmd.c which was not due to this commit.
>
> I'm sorry about this. I started fixing these compiler errors but was too
> slow :(
> Btw which compiler do you use ? My gcc 2.95 didn't output these warnings.

GCC 3.2 (on SuSE 8.1)

> In my last mail i mentioned another problem, apr_collapse_spaces removes
> all spaces, not only the leading and trailing ones. So here is a patch
> which fixes it and adds some more tests.

Thanks. Committed (with slight changes) in r7358.

But what about other special characters: equals sign, semicolon, spaces at the beginning or end of the value, etc.? We really need a general quoting mechanism. I think somebody mentioned that there is one in APR somewhere.

- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Oct 8 19:18:41 2003