[PATCH] allow T_INLINE_HTML before T_NAMESPACE
| From: | Lucas Stephanou | Date: | Fri, 12 Sep 2008 20:50:47 +0000 |
| Subject: | [PATCH] allow T_INLINE_HTML before T_NAMESPACE | ||
| References: | 1 2 3 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-40488@lists.php.net to get a copy of this message | ||
My first msg in this list
And I agree with you David.
Mix html and namespace is very very ugly.
2008/9/12 David Coallier <davidc@php.net>:
> 2008/9/12 Greg Beaver <greg@chiaraquartet.net>:
>> Hi,
>>
>> This is a simple patch that allows files like this:
>>
>> main.php:
>>
>> <html>
>> <head>
>> <title>template example</title>
>> </head>
>> </html>
>> <body>
>> <?php
>> namespace my::template;
>> // stuff
>> ?>
>> </body>
>>
>
> Is it me or this doesn't look really clean? I have a clear idea that
> namespaces are there to add an extra structural layer to the code and
> not to be randomly used in some HTML/PHP/Javascript mixing.
>
> Since the current namespace implementation doesn't want to have
> multiple namespaces per file, I think we should restrict a namespace
> per file to... a namespace per file, and nothing else. I can't imagine
> taking over someone's code who had declared a namespace under is
> footer template.
>
> Perhaps I am missing something but that would be bloody ugly in the end :)
>
> --
> Slan,
> David
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>
--
Lucas Stephanou
--
Lucas Stephanou