Re: cvs: php4 / CODING_STANDARDS
| From: | Yasuo Ohgaki | Date: | Sun, 08 Sep 2002 23:11:29 +0000 |
| Subject: | Re: cvs: php4 / CODING_STANDARDS | ||
| References: | 1 2 | Groups: | php.dev |
| Request: | Send a blank email to php-dev+get-88083@lists.php.net to get a copy of this message | ||
derick@php.net wrote:
On Sun, 8 Sep 2002, Yasuo Ohgaki wrote:I asked ZEND_* and PHP_* macro usage long time ago. There were sevral replys, but no objection for this [7] Use ZEND_* macros instead of PHP_* macros. Use of PHP_* macros is notyohgaki Sun Sep 8 18:38:57 2002 EDTerm, where was this decided? The whole source uses PHP_FUNCTION... [derick@kossu php-4.3.0dev]$ grep -r PHP_FUNCTION * | wc -l 5990 [derick@kossu php-4.3.0dev]$ grep -r ZEND_FUNCTION * | wc -lModified files: /php4 CODING_STANDARDS Log:It seems we're better to mention about missing functions. PHP_FUNCTION -> ZEND_FUNCTION217So I'd rather see those other functions converted to PHP_FUNCTION (execpt the zend builtin stuff of course).
recommended. Since most of the PHP_* macros are ZEND_* macro aliases, using
the PHP_* macros makes browsing the source code with a tag search harder.
--
Yasuo Ohgaki