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

Re: [PATCH] Fix issue #2068

From: Senthil Kumaran S <senthil_at_collab.net>
Date: Tue, 17 Mar 2009 17:02:25 +0530

David Glasser wrote:
> On Thu, Mar 12, 2009 at 3:03 AM, Bert Huijben <rhuijben_at_sharpsvn.net> wrote:
>> Maybe we should use a double semicolon as escape sequence for passing a
>> semicolon. I don't like the idea of breaking all old config files, registry
>> settings, etc..
>>
>> This fix only removes the semicolon problem, but it breaks putting |
>> characters in properties.
>
> That's a great suggestion, since in the previous format, any double
> semicolon was completely redundant.

As per above suggestion, I am sending this updated patch. Let me know if it
needs any change, if not I shall commit it tomorrow.

[[[
Fix issue #2068.

* subversion/libsvn_client/add.c
  (auto_props_enumerator): Add logic to escape double semicolons.

Suggested by: rhuijben
]]]

Thank You.

-- 
Senthil Kumaran S
http://www.stylesen.org/
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1340243

Received on 2009-03-17 12:32:43 CET

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