[PHP-CVS] RE: [PHP-DOC] Re: [PHP-CVS] [PHP-DOC] [PHP-CVS] svn: /php/php-src/branches/PHP_5_4/ UPGRADING

[Customer Service]

OK


From: Pierre Joye [pierre@gmail.com]
Sent: Tuesday, November 22, 2011 12:12 PM
To: Philip Olson
Cc: Hannes Magnusson; Peter Cowburn; PHP Documentation ML; Pierrick Charron; 
php-cvs@lists.php.net
Subject: [PHP-DOC] Re: [PHP-CVS] [PHP-DOC] [PHP-CVS] svn: 
/php/php-src/branches/PHP_5_4/ UPGRADING

On Tue, Nov 22, 2011 at 7:04 PM, Philip Olson  wrote:

> I cannot find a discussion or decision. An old patch was applied that
> lacked an associated RFC. People later noticed the E_DEPRECATED error,
> so it was removed. That's how I see it, at least,


We removed it because it makes no sense to have such a noise here,
that's not very developer friendly. But the function remains
deprecated and the related feature has been removed. Does that not
make sense to you to deprecate (in the doc only here, for the reason I
explained in here) a function related to a dead feature? If not then
I'm out of argument, sorry.

--
Pierre

@pierrejoye | http://blog.thepimp.net | http://www.libgd.org
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

[PHP-CVS] RE: [PHP-DOC] Re: [PHP-CVS] [PHP-DOC] [PHP-CVS] svn: /php/php-src/branches/PHP_5_4/ UPGRADING

[Customer Service]

OK


From: Philip Olson [phi...@roshambo.org]
Sent: Tuesday, November 22, 2011 12:04 PM
To: Pierre Joye
Cc: Hannes Magnusson; Peter Cowburn; PHP Documentation ML; Pierrick Charron; 
php-cvs@lists.php.net
Subject: [PHP-DOC] Re: [PHP-CVS] [PHP-DOC] [PHP-CVS] svn: 
/php/php-src/branches/PHP_5_4/ UPGRADING

On Nov 22, 2011, at 9:35 AM, Pierre Joye wrote:

> On Tue, Nov 22, 2011 at 6:28 PM, Hannes Magnusson
>  wrote:
>> On Tue, Nov 22, 2011 at 18:19, Pierre Joye  wrote:
>>> On Tue, Nov 22, 2011 at 5:56 PM, Philip Olson  wrote:

 I do not think they are deprecated, nor think they should be documented as 
 such.
>>>
>>> They are but we removed the warnings to ease migration.
>>>
>>
>> No they are not deprecated.
>> The feature itself is deprecated,
>
> The feature is removed in 5.4, not deprecated. So the function to
> check if MQ is enabled or not has been kept as convenience only for
> now and is deprecated. Please refer to the discussion about that on
> the internals mailing list. We only agreed on removing the warnings as
> it makes no sense to be noisy in this case.

I cannot find a discussion or decision. An old patch was applied that
lacked an associated RFC. People later noticed the E_DEPRECATED error,
so it was removed. That's how I see it, at least, so I do not feel
comfortable deprecating these. We've already made this sort of mistake
in the past (e.g., $str{42}), so we should not do it again until this
topic is clarified by the dev team.

Regards,
Philip
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php