Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:49725
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.82.170 as permitted sender)
DomainKey-Status: bad
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:in-reply-to:references:date:message-id:subject:from:to
         :cc:content-type:content-transfer-encoding;
        b=o6dbG1KWpPqPlHWVEGFJLIP44YW8szWzBgVtD3zmgJfXu4JHliNA2WSlb8ITz6ZYUn
         DI8Uc7BZs16eX8E9Hj/3j29pxnVrfOU6lJ2R5bztpuMnkMBP/zwiLIBzG7k7cA1nEsl2
         v587dp0eeBF5XV+GJGqtoHz8NQDWujU9raYL8=
MIME-Version: 1.0
In-Reply-To: <4C926574.8030805@sugarcrm.com>
References: <AANLkTimuWDV=9AA63_9M6cQSp9riCPE214sUNdoV3tQn@mail.gmail.com>
	<4C873C0F.1010200@zend.com>
	<AANLkTimdCnW3ABq5QSnBa78H14e2LPBa=Kt2txU0sTD4@mail.gmail.com>
	<4C879613.7090709@zend.com>
	<AANLkTim-dHYyJxX2R3-PBVqqa3sv3+zKYr7zw7QvyngJ@mail.gmail.com>
	<4C887D2B.2000605@zend.com>
	<4C8AC526.7000505@sugarcrm.com>
	<AANLkTikpCGj2zVNQmC1gUWaRSpsQD=qNrETi90wj_No7@mail.gmail.com>
	<4C8B6168.30504@mohiva.com>
	<AANLkTinuD_Aga-CbsbSq-23MTyOLPGrBEswTm=JtBY0H@mail.gmail.com>
	<4C8BC81E.8000605@sugarcrm.com>
	<AANLkTinFSqCFUE3P5XvjJPwUGZqYrQR_9VHAXAWSL=rv@mail.gmail.com>
	<7.0.1.0.2.20100913145703.0d226d90@zend.com>
	<alpine.DEB.2.00.1009161454590.24012@kossu.derickrethans.nl>
	<op.vi4pwzpsidpuyk@cataphract-old.dulce.lo.geleia.net>
	<AANLkTimwXKohy_ai1+ZWsUghkwgzcL=kT8dBi25oGcrZ@mail.gmail.com>
	<4C926574.8030805@sugarcrm.com>
Date: Thu, 16 Sep 2010 16:03:03 -0300
Message-ID: <AANLkTim7DWrTQ6D_SaJMrc5KqN8yYBAy+YKoOdnH1aFO@mail.gmail.com>
To: Stas Malyshev <smalyshev@sugarcrm.com>
Cc: Gustavo Lopes <glopes@nebm.ist.utl.pt>, Derick Rethans <derick@php.net>, 
	"internals@lists.php.net" <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] Re: PHP Annotations RFC + Patch
From: guilhermeblanco@gmail.com (Guilherme Blanco)

Hi Stas,

On Thu, Sep 16, 2010 at 3:44 PM, Stas Malyshev <smalyshev@sugarcrm.com> wro=
te:
> Hi!
>
>> Again, we should not consider docblock mainly because I think
>> adding/removing comments of your code should NEVER modify the overall
>> functionality of your application.
>
> It's a circular argument - "we can't use docblocks because docblocks
> shouldn't be used". They are not "comments" if you don't use them as
> comments.

I mean that any code packer can degrade the the functionality of your
app. Example:

class FakeTest extends \PHPUnit_Framework_TestCase
{
    /**
     * @expectedException RuntimeException
     */
    public function testShouldThrowException()
    {
        throw new \RuntimeException('FakeTest failed.');
    }
}

Would become:

class FakeTest extends \PHPUnit_Framework_TestCase
{
    public function testShouldThrowException()
    {
        throw new \RuntimeException('FakeTest failed.');
    }
}

And your test suite has now one failure.
As I said previously, comments are human readable data ONLY and should
not be shared between your code functionality and documentation.
Otherwise, let's rename the concept of docblock (documentation block)
to metadata block.

IMHO, it is conceptually wrong that by removing a comment from your
code may affect how your application works. At most, it should be less
documented, nothing else.

>
>> That said, docblock is no option. Now PLEASE let's stop arguing for
>> nothing and vote?
>
> I do not think we should stop discussing some topic just because somebody
> has made up his mind. Voting is fine, but "OK, I have made my mind,
> everybody shut up and vote" is not.

I didn't mean we should stop the discussion. I meant that like many
others over the years lead to nowhere if we don't take the correct
action: vote.
We can still continue the thread here, I just wanted people to stop
arguing about nothing (or even deviating the thread from its original
subject) without actually make any clever point.

>
>> I'd recommend that since syntax is not ideal, we should not vote for
>> the complete patch but vote for the functionality.
>>
>> So the question to be answered is: Should PHP support Annotations?
>>
>
> WTF is "Annotations"? We didn't define it yet. Should PHP support
> Fooblahblations? Sure, I'm +1 on it, let's vote.

I would point the wikipedia link, but it contains very weird
description. "In the year 2065, aliens rule the Earth. Why? Because
Noah Katz took over the world with the help of his alien friends. He
calls himself a king, but a warrior among the rebelling people is
about to rise up against the alien empire to reclaim the Earth for the
human race. This warrior is the all-powerful Zeus of Mount Olympus.
The Greek gods are angry, and will not have their world taken by some
people that possibly come from another universe. The end of the alien
empire will soon come."

So I'd recommend (although I didn't like how it was written) the Java
Annotation wikipedia page:
http://en.wikipedia.org/wiki/Java_annotation

> --
> Stanislav Malyshev, Software Architect
> SugarCRM: http://www.sugarcrm.com/
> (408)454-6900 ext. 227
>



--=20
Guilherme Blanco
Mobile: +55 (16) 9215-8480
MSN: guilhermeblanco@hotmail.com
S=C3=A3o Paulo - SP/Brazil