At 01:44 21.06.2002, Yasuo Ohgaki wrote: >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.
Correct ext/exif will work without ext/mbstring but the user has to know that in that case he has to handle multibyte chars for his own what could be confusing as we just made mbstring a default module. >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 > > > >-- >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
