Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:46708
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain lerdorf.com from 209.85.220.227 cause and error)
Message-ID: <4B4DFF46.7060908@lerdorf.com>
Date: Wed, 13 Jan 2010 09:13:42 -0800
User-Agent: Thunderbird 2.0.0.23 (X11/20090817)
MIME-Version: 1.0
To: Raphael Geissert <geissert@debian.org>
CC: internals@lists.php.net
References: <hij8pj$vbj$1@ger.gmane.org> <4B4D2883.5040703@lerdorf.com> <hikuja$3qs$1@ger.gmane.org>
In-Reply-To: <hikuja$3qs$1@ger.gmane.org>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV]  Throwing an E_DEPRECATED for short_open_tag
From: rasmus@lerdorf.com (Rasmus Lerdorf)

Raphael Geissert wrote:
> Rasmus Lerdorf wrote:
> 
>> Raphael Geissert wrote:
>>> However, we would like to contribute in the quest to make applications
>>> stop using short_open_tag. To do so, we have decided to throw an
>>> E_DEPRECATED warning when an application makes use of short_open_tag. The
>>> current implementation can be found at [1].
>>>
>>> How does this sound?
>> We have no plans to deprecate the short open tag.  So throwing an
>> E_DEPRECATED would not be appropriate.
>>
> 
> True. An idea I just had was to continue throwing E_DEPRECATED (or a warning 
> with a most appropriate name, if one is added) iff short tags are used 
> without explicitly enabling them at runtime.
> 
> I haven't looked at how the short_open_tag runtime switch was implemented so 
> I don't know if it's possible or not.
> 
> This would for example make the following code throw a warning:
> 
> foo.php
> <?
> // ...
> ?>
> 
> But not this one:
> 
> bar.php
> <?php
> // ...
> include("./foo.php");
> ?>
> 
> Does that sound more reasonable? or do you have another idea?

Then we have to have logic to differentiate <? from <?xml, for example.
 It would make the parser quite a bit more complex if we have to start
parsing stuff outside of the active open tags.  Not sure it is worth the
effort nor the extra overhead.

-Rasmus