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

Re: log --search test failures on trunk and 1.8.x

From: Ivan Zhakov <ivan_at_visualsvn.com>
Date: Sat, 20 Apr 2013 20:04:59 +0400

On Sat, Apr 20, 2013 at 7:54 PM, Branko Čibej <brane_at_wandisco.com> wrote:
> On 20.04.2013 17:42, Stefan Sperling wrote:
>> On Sat, Apr 20, 2013 at 07:26:06PM +0400, Ivan Zhakov wrote:
>>> On Sat, Apr 20, 2013 at 2:27 PM, Stefan Sperling <stsp_at_apache.org> wrote:
>>>> How is lower-casing a multi-byte UTF-8 character going to help?
>>>> Won't the lower-case equivalent still be a multibyte character
>>>> and trigger the overflow check in Visual Studio?
>>>>
>>> Because in this case we can call apr_fnmatch without
>>> APR_FNM_CASE_BLIND, avoid tolower calls and get case-insensitive
>>> match.
>> Are we going to implement our own code to transform a string
>> to lower case? Or do we already have such functions for UTF-8 strings?
>
No, we don't have such functions. We can try to convert UTF-8 string
to wchar_t and use towlower(), but probably it's overkill.

> We aren't and we don't, on trunk; but the wc-collate-path branch does
> have code that can do Unicode-aware case folding through utf8proc.
>
It looks like a best solution.

> It's always been wrong to use tolower for case-insensitive comparisons
> anyway, but we've tended to ignore that.
>
Yes, I know about this but I think we can currently ignore it.

-- 
Ivan Zhakov
CTO | VisualSVN | http://www.visualsvn.com
Received on 2013-04-20 18:05:51 CEST

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.