Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:9111
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Message-ID: <021b01c42188$b8885db0$8a02a8c0@tron>
To: "Derick Rethans" <derick@php.net>, "Andi Gutmans" <andi@zend.com>
Cc: <internals@lists.php.net>
References: <5.1.0.14.2.20040413183654.039c6b40@127.0.0.1> <Pine.LNX.4.58.0404132029070.1976@localhost>
Date: Tue, 13 Apr 2004 19:54:14 +0100
Organization: The Brain Room Ltd.
MIME-Version: 1.0
Content-Type: text/plain;
	charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] RC2RC1
From: wez@thebrainroom.com ("Wez Furlong")

+1 (and it's already on by default in the win32 build).

--Wez.

> I've one other thing to propose, and that is enabling iconv by default.
> Now you might ask why iconv and not mbstring? Well, that's easy as the
> author of mbstring (Moriyoshi) thinks of moving it to PECL, out of the
> core. Why I think iconv should (and can) be enabled by default is:
> 
> - libxml2 which we 'require', requires a working iconv implementation
>   (for Linux this is embedded in glibc, for FreeBSD this is the iconv
>   library, other Operating Systems of course have some form of
>   implementation too).
> 
> - conversion between charsets is getting more and more of a problem, and
>   to support some of the larger multi-language frameworks a bit better,
>   having an implementation in C for this is a very good thing. (There
>   are already implementations of conversions out in PHP, which is
>   ofcourse awfully slow)
> 
> - I am not in favor of enabling things by default *at all*, but I do
>   think it's worthwhile to do for iconv.