Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:18501
Return-Path: <preinheimer@gmail.com>
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Delivered-To: mailing list internals@lists.php.net
Received: (qmail 20593 invoked by uid 1010); 28 Aug 2005 04:42:56 -0000
Delivered-To: ezmlm-scan-internals@lists.php.net
Delivered-To: ezmlm-internals@lists.php.net
Received: (qmail 20578 invoked from network); 28 Aug 2005 04:42:56 -0000
Received: from unknown (HELO lists.php.net) (127.0.0.1)
  by localhost with SMTP; 28 Aug 2005 04:42:56 -0000
X-Host-Fingerprint: 66.249.82.197 xproxy.gmail.com  
Received: from ([66.249.82.197:3362] helo=xproxy.gmail.com)
	by pb1.pair.com (ecelerity 2.0 beta r(6323M)) with SMTP
	id 64/FA-28235-FC041134 for <internals@lists.php.net>; Sun, 28 Aug 2005 00:42:55 -0400
Received: by xproxy.gmail.com with SMTP id h29so472989wxd
	for <internals@lists.php.net>; Sat, 27 Aug 2005 21:42:52 -0700 (PDT)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
	s=beta; d=gmail.com;
	h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references;
	b=lg/OWyFl+P4MFVVkdqP9n3GNAA/khromfBZ2v+KEzU4fSx4MRCRVyzyEE994OaB1X8foBcLC0X4YhHf8JFolYWZyH/JGxCG9+YZ0BfSNKUzhdpvNIxHfmjwNa689iDCCQtU9McVMSMGRHDjYqFSjinASQ3nqijy+u+7Q6pMPQVw=
Received: by 10.70.92.8 with SMTP id p8mr74723wxb;
	Sat, 27 Aug 2005 21:42:52 -0700 (PDT)
Received: by 10.70.104.10 with HTTP; Sat, 27 Aug 2005 21:42:52 -0700 (PDT)
Message-ID: <6ec19ec7050827214249925901@mail.gmail.com>
Date: Sun, 28 Aug 2005 00:42:52 -0400
To: internals@lists.php.net
In-Reply-To: <129301c5ab69$71e31370$9cd96751@foxbox>
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline
References: <42FCE0E4.604@lerdorf.com> <20050827191013.GA13783@panix.com>
	<4310BBD1.3030408@prohost.org>
	<005101c5ab53$1d977e70$6c051fac@lighthammer>
	<129301c5ab69$71e31370$9cd96751@foxbox>
Subject: Re: [PHP-DEV] PHP 6.0 Wishlist
From: preinheimer@gmail.com (Paul Reinheimer)

My prefrerence would be for the interpreter to look for "<?\w" when
short tags is enabled (that was brace, question mark, white space), I
don't think i've ever come across an instance when a space or line
break wasn't used. I don't have a huge code repository handy but are
there any good counter examples live?

It seems that checking for whitespace (or the long tag) would resolve
all of the <?xml* issues, while retaining the majority of BC, and
hopefully allow PHP to do what it does best, work the way developers
expect it to.



paul


On 8/27/05, Steph <steph@zend.com> wrote:
> Yep.  I saw this happening during the PHP 5 coding contest we held last y=
ear
> at Zend - some people used short open tags so I had to have that 'on' in =
my
> php.ini to test, but having it 'on' screwed the entries that used XML, so=
 I
> was forever switching short open tags off and on.
>=20
> I was supposed to prove that there was a problem in that area, at the tim=
e,
> having opened my mouth over this issue.
>=20
> <regret>I didn't actually make time to do so</regret>
>=20
> - but yes, I believe it's an issue.
>=20
> ----- Original Message -----
> From: "Sara Golemon" <pollita@php.net>
> To: <internals@lists.php.net>
> Sent: Sunday, August 28, 2005 12:02 AM
> Subject: Re: [PHP-DEV] PHP 6.0 Wishlist
>=20
>=20
> > >> It would be nice if PHP could not get tripped up by "<?xml" tags whe=
n
> > >> short_tags is on.
> > >>
> > > Already included, you may have seen this feature used before, <?php :=
-)
> > >
> > I gather he meant making the scanner intelligent enough to see that the=
 <?
> > is followed by 'xml' and just ignoring the fact that short open tags is
> on.
> > I wouldn't expect to see it happen (or want it to), but if he wants it,
> > here's a sloppy implementation:
> >
> > Index: Zend/zend_language_scanner.l
> > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
> > RCS file: /repository/ZendEngine2/zend_language_scanner.l,v
> > retrieving revision 1.131
> > diff -u -r1.131 zend_language_scanner.l
> > --- Zend/zend_language_scanner.l        3 Aug 2005 13:30:54 -0000
> > 1.131
> > +++ Zend/zend_language_scanner.l        27 Aug 2005 21:55:36 -0000
> > @@ -1379,6 +1379,12 @@
> >         }
> >  }
> >
> > +<INITIAL>"<?xml"([ \t]|{NEWLINE}) {
> > +       zendlval->value.str.val =3D (char *) estrndup(yytext, yyleng);
> > +       zendlval->value.str.len =3D yyleng;
> > +       zendlval->type =3D IS_STRING;
> > +       return T_INLINE_HTML;
> > +}
> >
> >  <INITIAL>"<%=3D"|"<?=3D" {
> >         if ((yytext[1]=3D=3D'%' && CG(asp_tags)) || (yytext[1]=3D=3D'?'=
 &&
> > CG(short_tags))) {
> >
> > --
> > PHP Internals - PHP Runtime Development Mailing List
> > To unsubscribe, visit: http://www.php.net/unsub.php
> >
>=20
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>=20
>=20


--=20
Paul Reinheimer
Zend Certified Engineer