Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:87983
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.213.169 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <55E4B9AA.9060906@dasprids.de>
References: <55E4B9AA.9060906@dasprids.de>
Date: Tue, 1 Sep 2015 10:54:54 +0200
Message-ID: <CAB6YZuyMMwSywq+ApS4peeMxYCYwQxNNLDsKu369mavMf717bg@mail.gmail.com>
To: "Ben Scholzen 'DASPRiD'" <mail@dasprids.de>
Cc: PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [PHP-DEV] Generic classes and methods RFC
From: pajousek@gmail.com (=?UTF-8?Q?Pavel_Kou=C5=99il?=)

On Mon, Aug 31, 2015 at 10:31 PM, Ben Scholzen 'DASPRiD'
<mail@dasprids.de> wrote:
> Hello,
>
> I've written up an RFC for supporting generic classes and methods in PHP,
> and I'd love to hear your thoughts about it.
>
> https://wiki.php.net/rfc/generics
>
> Cheers,
> --
> Ben Scholzen 'DASPRiD'
> Community Review Team Member | mail@dasprids.de
> Zend Framework               | http://www.dasprids.de
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>

Hello,

I would love to see generics in PHP, but I have a few questions for your RFC.

class Bazz<Foo : \Bar>

Why did you use this syntax for forcing extending/implementation? I
know this is the syntax C# uses, but it doesn't fit PHP. I think it
should be "extends" or "implements", because in PHP : has a different
meaning.

Also, how do you specify multiple constraints?

Do you have any idea how it would work internally and if there was
some performance hit while using generics, and if it would also slow
down existing apps that don't use generics?

I hope this RFC will have positive comments, and someone who is
capable of writing C will help you out with it so there's a patch for
people to review - because I'm not sure if this RFC can get anywhere
without a patch. :(

Regards
Pavel Kouril