Re: [PATCH v2 1/6] Add string comparison functions that respect the ignore_case variable.

> On Sun, Oct 3, 2010 at 04:32, Joshua Jensen<jjensen@workspacewhiz.com>  wrote
>> +int fnmatch_casefold(const char *pattern, const char *string, int flags)
>> +{
>> +       char lowerPatternBuf[MAX_PATH];
>> +       char lowerStringBuf[MAX_PATH];
>> +       char* lowerPattern;
>> +       char* lowerString;
>> +       size_t patternLen;
>> +       size_t stringLen;
>> +       char* out;
>> +       int ret;
>> +
>> +       /*
>> +        * Use the provided stack buffer, if possible.  If the string is too
>> +        * large, allocate buffer space.
>> +        */
>> +       patternLen = strlen(pattern);
>> +       if (patternLen + 1>  sizeof(lowerPatternBuf))
>> +               lowerPattern = xmalloc(patternLen + 1);
>> +       else
>> +               lowerPattern = lowerPatternBuf;
>> +
>> +       stringLen = strlen(string);
>> +       if (stringLen + 1>  sizeof(lowerStringBuf))
>> +               lowerString = xmalloc(stringLen + 1);
>> +       else
>> +               lowerString = lowerStringBuf;
>> +
>> +       /* Make the pattern and string lowercase to pass to fnmatch. */
>> +       for (out = lowerPattern; *pattern; ++out, ++pattern)
>> +               *out = tolower(*pattern);
>> +       *out = 0;
>> +
>> +       for (out = lowerString; *string; ++out, ++string)
>> +               *out = tolower(*string);
>> +       *out = 0;
>> +
>> +       ret = fnmatch(lowerPattern, lowerString, flags);
>> +
>> +       /* Free the pattern or string if it was allocated. */
>> +       if (lowerPattern != lowerPatternBuf)
>> +               free(lowerPattern);
>> +       if (lowerString != lowerStringBuf)
>> +               free(lowerString);
>> +       return ret;
>> +}
>> +
>> +int fnmatch_icase(const char *pattern, const char *string, int flags)
>> +{
>> +       return ignore_case ? fnmatch_casefold(pattern, string, flags) : fnmatch(pattern, string, flags);
>> +}
>
> I liked v1 of this patch better, although it obviously had portability
> issues. But I think it would be better to handle this with:
>
>      #ifndef FNM_CASEFOLD
>      int fnmatch_casefold(const char *pattern, const char *string, int flags)
>      {
>          ...
>      }
>      #endf
>
>      int fnmatch_icase(const char *pattern, const char *string, int flags)
>      {
>      #ifndef FNM_CASEFOLD
>             return ignore_case ? fnmatch_casefold(pattern, string,
> flags) : fnmatch(pattern, string, flags);
>      #else
>              return fnmatch(pattern, string, flags | (ignore_case ?
> FNM_CASEFOLD : 0));
>      #endif
>      }
>
> Or simply use fnmatch(..., FNM_CASEFOLD) everywhere and include
> compat/fnmatch/* on platforms like Solaris that don't have the GNU
> extension.