Re: Re: cvs: php4 /ext/mbstring mbstring.c

Brian France wrote:

Ok, what does exif need from mbstring, code wise? Taking a look at it I don't see any references to mbstring in the code.

exif needs mbstring to handle multibyte chars. IMO, removing mbstring from PHP is ok as long as user knows what they are doing. We can remove other internal modules, such as session module also. -- Yasuo Ohgaki

Thanks, Brian At 8:27 PM +0200 6/20/02, Marcus Börger wrote:

You will break ext/exif when removing mbstrings current integration At 04:17 20.06.2002, you wrote:

I am testing a patch that allows mbstring to be built as a shared extension instead of static. This would allow removing any reference to mbstring from the core PHP code and making mbstring movable out of the ext directory possible. Not that moving it out needs to happen, but as much talk about moving things out of the ext directory I thought it would be a selling point :-) The patch basically renames php_treat_data to php_treat_data_default, creates a function pointer called php_treat_data that is defaulted to php_treat_data_default, removes all mbstrings references in php_main.h and makes mbstring.c change the php_treat_data to mbstr_treat_data in the INIT function and restores its value in SHUTDOWN. Would you be interested in it? Thanks, Brian At 10:36 AM +0900 6/20/02, Yasuo Ohgaki wrote:

Marcus BöRger wrote:

helly           Wed Jun 19 17:55:46 2002 EDT

Modified files:

    /php4/ext/mbstring  mbstring.c   Log:

correct handling/generating of php_mbstr_default_identify_list

As I posted before, I'll try merge changes made in php4/ext/mbstring to PHP i18n project repository's source. Please note that I'm not promising that for sure. mbstring source has been modified a lot. Please check http://cvs.sourceforge.jp/cgi-bin/viewcvs.cgi/php-i18n/ to avoid duplicated work or to make sure your work is included. -- Yasuo Ohgaki -- PHP Development Mailing List <http://www.php.net/> To unsubscribe, visit: http://www.php.net/unsub.php

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