Re: cvs: php-src /ext/hash CREDITS config.m4 hash.c /ext/hash/tests mhash_001.phpt mhash_002.phpt mhash_003.phpt skip_mhash.inc /ext/mhash config.m4
mhash.c mhash.dsp php_mhash.h /ext/mhash/tests 001.phpt 002.phpt 003.phpt skip.inc
| From: | Pierre Joye | Date: | Sun, 17 Aug 2008 20:32:33 +0000 |
| Subject: | Re: cvs: php-src /ext/hash CREDITS config.m4 hash.c /ext/hash/tests mhash_001.phpt mhash_002.phpt mhash_003.phpt skip_mhash.inc /ext/mhash config.m4 mhash.c mhash.dsp php_mhash.h /ext/mhash/tests 001.phpt 002.phpt 003.phpt skip.inc |
||
| References: | 1 2 3 4 5 6 7 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-52449@lists.php.net to get a copy of this message | ||
On Sun, Aug 17, 2008 at 9:26 PM, Jani Taskinen <jani.taskinen@sci.fi> wrote:
> Pierre Joye kirjoitti:
>>
>> hi,
>>
>> On Sun, Aug 17, 2008 at 2:09 PM, Jani Taskinen <jani.taskinen@sci.fi>
>> wrote:
>>
>>> You don't need to have multiple PHP_NEW_EXTENSION lines in config*.m4's
>>> to
>>> enable multiple extensions and is not really useful either since you
>>> break
>>> the ext_shared stuff, etc. If you do that anywhere, you're doing
>>> something
>>> wrong.
>>
>> I was not saying that, I only stated why the other ext has been kept.
>> If you can do it in a clean(er) way, please go ahead :)
>>
>> Cheers,
>
> It's simply ridiculous to keep a dummy extension JUST that someone can do
> extension_exists("mhash")..
I agree, but you know my opinion about such BC decision :)
--
Pierre
http://blog.thepimp.net | http://www.libgd.org