Hi, Nikita, Xinchen and I have sit down together this morning to solve a couple of naming issues, those having caused some flames, trolls and FUDs in the last couple of days. The good news is we have found a consensus that solves our concerns, consistency and reduce the amount of changes in existing code.
Please not that this mail is not my personal taste or wishes, but the results of a refreshing, constructive and good discussions with Xinchen and Nikita earlier this morning. Summary: zend_uint > int32_t zend_size_t > size_t zend_int_t > zend_long IS_INT > IS_LONG Z_IVAL > Z_LVAL Z_STRSIZE > Z_STRLEN zend_str_* > zend_string_* STR_* macros > droped Details: We have a header available by default to define c9* stdint, see main/php_stdint.h. This header ensures consistency accross platforms and that the size of each type is correct. It is important to keep that in mind while reading the rest of this mail. * zend_uint zend_uint is used exclusevely in the engine (maybe some debugger or engine related extensions) and has been around for a while. We like to drop it and use int32_t instead anywhere where zend_uint is used. Using int32_t is good because this will ensure that we use the correct type even on ILP64 - zend_uint would currently totally blow up on those systems. *zend_size_t Given that size_t is a standard type, adding zend_size_t makes little sense. * zend_int_t, zend_long, IS_INT/IS_LONG One of my main concerns is consistency between APIs and macros names and the underlying types. A solution to this problem was quite obvious (only not on Friday ;). zend_int_t will be renamed to zend_long, IS_INT and Z_IVAL will renamed to their old names, IS_LONG and Z_LVAL, respectively. * Strings First change, Z_STRSIZE will be renamed to Z_STRLEN (old name). We also like to drop STR_ macros and rely directly on the zend_string_* APIs. While doing at it the zend_string APIs will be prefix zend_string and not zend_str_* as it is now, for the same reason: During the NG works and the ports of many extensions, numerous bugs appear due to bad usage of STR_* macros with either char* or other char/string related types. Making the name more obvious will help. PS: IRC log available if anyone likes to double check it, #php.pecl Cheers, -- Pierre @pierrejoye | http://www.libgd.org -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
