cURL / Mailing Lists / curl-library / Single Mail


Re: [PATCH] New function Curl_fnmatch() added

From: Daniel Stenberg <>
Date: Mon, 26 Apr 2010 13:03:19 +0200 (CEST)

On Mon, 26 Apr 2010, Pavel Raiskup wrote:

> I'd like to propose you this function which should be used as the default
> matching function for CURLOPT_WILDCARDMATCH (implemented ASAP). It could be
> used for other features in future too.

It doesn't make much sense for us to add this funtion independently of your
other work, as we don't add code that "might be good to have in the future" by
policy. If there's code that needs this, then we add it.

I appreciate your work and once the wildcardmatching magic goes in I don't see
any problems with adding this as well. I'm also very happy to see that you
made such an elegant test case for the test suite that verifies the fnmatch

> POSIX fnmatch(3) vs. Curl_fnmatch()

So they are otherwise 100% expression compatible?

List admin:
Received on 2010-04-26