It's been a few days since the last comments regarding the fgetcsv() in PHP 
5.0 and I think it would be an opportune time to decide on a solution for the 
problem. Based on the various commentary we appear to have 3 alternatives 
that for better or worse would resolve the situation.

1) Leave things they way they are now, giving everyone a partial support for 
multibyte string in fgetcsv() and 2-3 performance decrease in all instances.

2) Backport fgetcsv() from 4.3.X tree, giving everyone a fast variant of the 
function and introduce a fully multibyte mb_fgetcsv() into mbstring extension 
based on the current fgetcsv() in PHP 5.0.

3) Put the php_mblen macro in php_string.h inside ifdef HAVE_MBSTRING, which 
would disable multibyte functionality of fgetcsv() for people without 
mbstring support in their installation. This would just about restore 
performance and at the same time leave multibyte support for people who need 
it (use mbstring extension).


My personal preference is option #2.

Ilia

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to