Andrei Zmievski <[EMAIL PROTECTED]> wrote: > On Wed, 13 Nov 2002, Marcus Börger wrote: > > Hatmud and Dan mentioned some comparison to other comparable changes. > > Which had negative side effects and generally Dan is known to be against > > enabling any module by default. > > > > For me the concern was that it may be enabled by default if it is stable. > > As Derick wrote this seems the same as the result we now have. > > The only difference is that for some reason some people now think mbstring > > isn't stable. I guess that came from the start of the discussion where some > > talk of the conference was reported, something like "function overloading > > could > > harm php". But function overloading was disabled by default i thought. > > As Derick (or someone else) mentioned, mbstring is a very volatile > extension, it seems. It receives several commits a day and we do not > know what kind of things it may break, on Linux, Windows, or other > platforms. Having multibyte functionality (and other stuff) is great, > but not at the cost of stability of the whole package.
Sorry for my repetitive change since the day I declared it had been done. But I've ever tried to clarify why the patch was needed, and how the patch works. Then I don't understand the reason you say *we* dunno what it may break, though it doesn't hurt anything else now as the unit tests goes, besides it's actually required by several extensions. If my effort isn't making any sense, I apologize. However, if you simply haven't tried to understand how this module works in relevance to the rest of the source codes, please reconsider it. Regards, Moriyoshi > -Andrei http://www.gravitonic.com/ > > Some people sell razors. Other people sell razor blades. Microsoft > sells shaving cream containing 5% hair growth hormone. > -- sigwinch, posting to Kuro5hin > > -- > PHP CVS Mailing List (http://www.php.net/) > To unsubscribe, visit: http://www.php.net/unsub.php > > -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
