Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:40115
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.44.30 as permitted sender)
DomainKey-Status: bad
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=subject:from:to:cc:in-reply-to:references:content-type:date
         :message-id:mime-version:x-mailer:content-transfer-encoding;
        b=vrZiRqrV4HT8Y6MGRojcomTQqKBskwpciNTS7WXn5A38bXq8hUzoij6bO4RdysCtSZ
         qgsxjP7s0JQpGv1aEZ6LcZrv1crTJhnsTKvYCFCwPsTo6d3J/EXE5A3lSOxEU+aEBI1i
         aSlb8xju0j027yJphMX+EIvYaIk4IP/Op1MjE=
To: Marcus Boerger <helly@php.net>
Cc: Hannes Magnusson <hannes.magnusson@gmail.com>, Lukas Kahwe Smith <mls@pooteeweet.org>, PHP Internals List <internals@lists.php.net>, Johannes =?ISO-8859-1?Q?Schl=FCter?= <johannes@php.net>
In-Reply-To: <322434106.20080827234712@marcus-boerger.de>
References: <64B4CECA-980F-4738-8B05-E7935741D7BB@pooteeweet.org>
	 <E02EE232-938B-4E51-AB55-728C639642FD@pooteeweet.org>
	 <7f3ed2c30808271132x7443ea3gc4df903a3a2c73ec@mail.gmail.com>
	 <322434106.20080827234712@marcus-boerger.de>
Content-Type: text/plain; charset=utf-8
Date: Wed, 27 Aug 2008 21:57:37 -0300
Message-ID: <1219885057.5961.13.camel@pena>
Mime-Version: 1.0
Content-Transfer-Encoding: 8bit
Subject: Re: [PHP-DEV] alpha2 scheduled
From: felipensp@gmail.com (Felipe Pena)

Hello Marcus,

Em Qua, 2008-08-27 às 23:47 +0200, Marcus Boerger escreveu:
> Hello Hannes,
> 
> Wednesday, August 27, 2008, 8:32:41 PM, you wrote:
> 
> > On Tue, Aug 26, 2008 at 18:27, Lukas Kahwe Smith <mls@pooteeweet.org> wrote:
> >> - Curly braces support for namespaces (Marcus, not ready for alpha2)
> 
> > Why not? I thought I saw a patch for this on the list recently (which
> > allowed both the current namespace foo; and namespace foo {}
> > syntaxes)..
> 
> It was an incomplete patch that simply allowed the syntax but did not
> prevent code between two namespaces.


That my old patch prevent the code between namespaces, but it only allow
the syntax using { }. Then, I've updated it, now it allows both ({}, ;)

Well, maybe it can be useful...

Results using the patch:

<?php
namespace a { 
namespace b {
}
}
Fatal error: Nested namespace is not allowed in %s on line %d

---------------------

<?php
namespace a { 
namespace b;
}
Fatal error: Nested namespace is not allowed in %s on line %d

---------------------

<?php
namespace a;
namespace b {}
namespace c;
print 1;
// OK, no error!


---------------------

<?php

namespace a;
namespace b {}
print 1;

Fatal error: This script contains namespaces, all other code must be
contained within namespace declarations in %s on line %d


---------------------

<?php

namespace a;
print 1;
namespace b {}
// OK, no error!


---------------------


<?php

print 1;
namespace a;

Fatal error: Namespace declaration statement has to be the very first
statement in the script in %s on line %d



http://felipe.ath.cx/diff/namespace-braces-5_3.diff

-- 
Regards,
Felipe Pena.