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

Re: [PATCH in progress] Ref. Issue#3781 Make parsing section names case sensitive

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Mon, 31 Jan 2011 10:19:23 -0500

On 01/31/2011 10:05 AM, Arwin Arni wrote:
> On Monday 31 January 2011 08:01 PM, C. Michael Pilato wrote:
>> On 01/31/2011 04:48 AM, Arwin Arni wrote:
>>> Should I add parameters for options / values / options& values to decide
>>> the case or shall I use a common parameter for all(sections, options&
>>> values) ?
>>
>> We generally try to avoid premature over-engineering, and at the moment, we
>> have no known need for the additional changes you speak of. Introducing
>> codepaths that we don't exercise can lead to surprise failures later when we
>> do.
> But I am excercising it!
>
> In libsvn_repos/authz.c. I'm using it to make the config population
> case-sensitive. (Issue #3781)

Unless I'm mistaken, your immediate need is to make *section* parsing
case-sensitive. Nothing about issue #3781 calls for toggle-able case
handling of options or values. *That's* the "additional changes" stuff I'm
referring to.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Received on 2011-01-31 16:19:59 CET

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.