Re: [PHP-DEV] [RFC] Better type names for int64 RFC

[email protected] (Dmitry Stogov)
Newsgroups php.internals
Message-ID <CA+9eiLsW4OBfj90mzZuW=HKstcLjfVbXQB5dfcm9sj8WrOF7zg@mail.gmail.com>
I think it's ok to keep zend_off_t and zend_size_t as is.

Thanks. Dmitry,


On Fri, Aug 22, 2014 at 4:09 PM, Anatol Belski <[email protected]>
wrote:

> Hi Nikita,
>
> On Fri, August 22, 2014 13:16, Nikita Popov wrote:
> > Hi internals!
> >
> >
> > Today the int64 RFC has been merged, despite objections regarding the
> > naming changes it introduces.
> >
> > As we were not given a chance to resolve this issue before the merge, a
> > short proposal has been created, which aims to revert all unnecessary
> > naming changes and instead use type names that are consistent with the
> > existing code base and expectations:
> >
> > https://wiki.php.net/rfc/better_type_names_for_int64
> >
> >
> > Due to the unexpected merge on short notice, with no chance for
> > discussion, this issue is blocking a number of other patches. As such I
> > ask if we can move through this RFC with a shortened cycle. I would not
> > appreciate having to wait three weeks to have a workable codebase again.
> >
> > Nikita
> >
> to be complete, the RFC probably should take in account several new
> identifiers which came in (zend_off_t, zend_stat_t and co.). As their
> naming scheme is probably somewhat different. I've just documented those
> http://git.php.net/?p=php-src.git;a=blob;f=UPGRADING.INTERNALS and
> continue.
>
> Regards
>
> Anatol
>
>
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.