+1 for option #2. Joël.
On Sun, Apr 18, 2010 at 11:58 PM, Adam Harvey <ahar...@php.net> wrote: > As at least some of you would already be aware, there's a > long-standing issue with using PHP in a Turkish or Azeri locale, > namely that case-insensitive lookups within the Zend engine (method > names, for example) fail on lookups involving upper-case I characters, > since lower-case I in those languages is ı instead of i (note the lack > of a dot). > > The long term plan for this, per bug #35050 and any number of > duplicates, was to deal with it in PHP 6. Since PHP 6 isn't going to > happen in its original form, I think we're going to need to revisit > how we want to deal with this. There's a patch linked in the bug from > Tomas Kuliavas and Marcus that fixes the problem by simply redefining > zend_tolower() to a simple locale-insensitive ASCII tolower() > function, which does fix the Turkish and Azeri locales. > > The potential breakage from this is that single-byte locales will no > longer get case-insensitive lookups of non-ASCII characters: for > example, somebody using fr_FR.ISO-8859-1 as a locale could no longer > call a method É() as é(). Since it doesn't break anything when using > multi-byte locales (which have never had case-insensitive lookups > anyway since the Zend Engine uses the single-byte tolower() > internally), my inclination would be to apply the patch on trunk and > document it as a BC issue. > > I've uploaded an updated version of Tomas's patch that applies cleanly > to trunk to http://www.adamharvey.name/patches/35050/zend_operators.c.diff > and a phpt file to test the fix to > http://www.adamharvey.name/patches/35050/bug35050.phpt. It's likely > that the test would require massaging before being committed to work > on Windows, but since I don't have a Windows development box readily > available and don't know a thing about how Windows implements locale > support, this would require help from someone familiar with the > platform. > > So: thoughts; concerns; alternate approaches? It would be nice to have > this sorted for PHP.next. > > Thanks, > > Adam > > -- > PHP Internals - PHP Runtime Development Mailing List > To unsubscribe, visit: http://www.php.net/unsub.php > > -- I do know everything, just not all at once. It's a virtual memory problem. -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
