Re: svn commit: r1383480 - in /subversion/trunk/subversion: svn/cl.h svn/log-cmd.c svn/main.c tests/cmdline/getopt_tests_data/svn_help_log_switch_stdout tests/cmdline/log_tests.py

On 20.09.2012 23:42, Stefan Sperling wrote:
> On Thu, Sep 20, 2012 at 02:30:30PM -0700, Bert Huijben wrote:
>> Fnmatch support is in apr(-util?) and has known access/performance patterns,
>> so we might be able to move the search handling to the server in the future
>> for performance reasons.
>>
>> Complete regular expression support would at least require using a new third
>> party library and moving it to the server is not really possible because
>> complete regular expression processing can be way to expensive.
> What he said. I was about to write the same response :)
>
> I *did* search for regex stuff in APR and other existing dependencies
> and then settled on fnmatch. httpd has a regex engine but it's not
> part of APR for some reason. I wouldn't have a problem using that
> in the client if it were available, but it isn't.

HTTPD uses PCRE, which has a BSD license. We could use it, but I'd
prefer to have a much better reason for doing so.

(e.g., "svn grep" could be such a reason)

-- Brane

-- 
Certified & Supported Apache Subversion Downloads:
http://www.wandisco.com/subversion/download