Re: [RFC][Discussion] Return Type Variance Checking
| From: | Dmitry Stogov | Date: | Fri, 28 Nov 2014 18:37:23 +0000 |
| Subject: | Re: [RFC][Discussion] Return Type Variance Checking | ||
| References: | 1 2 3 4 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-79285@lists.php.net to get a copy of this message | ||
Hi Levi,
if you remember, in my patch return_type was actually stored in
arg_info[-1].
it was mainly done for unification and to allow return type hinting for
internal functions (they already use arg_info[-1])
So the strictures may be compatible.
See https://gist.github.com/dstogov/8deb8b17e41c1a5abf88
I also thought about zend_string* usage in arg_info
Please review the patch if you missed it.
https://gist.github.com/dstogov/aa452a47ec30a8cb6eb5
The patches may be outdated, but they worked.
I may care about the final implementation if you agree with invariant
return type.
Thanks. Dmitry.
On Fri, Nov 28, 2014 at 7:51 PM, Levi Morrison <levim@php.net> wrote:
> > I prefer option (3) - invariant return types.
> > Actually, return type compatibility check should follow all the rules for
> > parameter type compatibility check (may be even reuse or share the code).
>
> Realistically there isn't much code to share, especially since the two
> structures are incompatible. That is, they are incompatible until
>
zend_string * is used in parameters (which I'm hopeful
> will happen).
>