Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:102341
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.82.42 as permitted sender)
MIME-Version: 1.0
References: <CAF+90c_L7S+EB8YxdNNpNZ7NhhYwCbwEw5_TSnSwUNN-zLVvCg@mail.gmail.com>
In-Reply-To: <CAF+90c_L7S+EB8YxdNNpNZ7NhhYwCbwEw5_TSnSwUNN-zLVvCg@mail.gmail.com>
Date: Thu, 21 Jun 2018 10:15:25 -0300
Message-ID: <CAP=4hC+s7tEwcZQ1R13BCmpWa4ioeJhM6wPMboBXW_g5VNvaPQ@mail.gmail.com>
To: Nikita Popov <nikita.ppv@gmail.com>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="00000000000024c9a1056f26b6dc"
Subject: Re: [PHP-DEV] [RFC] Typed Properties
From: carusogabriel34@gmail.com (Gabriel Caruso)

--00000000000024c9a1056f26b6dc
Content-Type: text/plain; charset="UTF-8"

Nikita, Bob

I'd love to thank you and everyone involved on this one. Types properties
on PHP is something that we tried before, but due to a number of issues, we
didn't make it. Unfortunately, I don't have the knowledge to help in the
implementation, but I'd like to drop some comments about the syntax.

As described in https://wiki.php.net/rfc/typed_properties_v2#syntax, I
think we should only allow
    class Example {
        public int $num;
        public string $name;
    }

Allow the `:` variant (`public $num: int;`) would bring a lot of doubts for
new users, and the `{visibility} {type} {name}` is prett consistence in
other languages.

Again, thanks for this amazing work, and let's make for the next PHP
version :)
-- 
Gabriel Caruso

--00000000000024c9a1056f26b6dc--