Re: zend_parse_parameters changes not fully documented in phpng-upgrading, PR to fix existing tools
| From: | Johannes Schlüter | Date: | Fri, 25 Sep 2015 12:18:12 +0000 |
| Subject: | Re: zend_parse_parameters changes not fully documented in phpng-upgrading, PR to fix existing tools | ||
| References: | 1 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-88496@lists.php.net to get a copy of this message | ||
Hi,
On Wed, 2015-09-23 at 15:39 -0500, Sean DuBois wrote:
> C.) Has there ever been a conversation about improving extension building so that authors
> don't have to go out of their way to catch these things? I would really like to see phpize
> kick out a Makefile that in the process of building/testing would raise alerts about this.
If you're using clang as compiler you can load this static analyzer
plugin: https://github.com/johannes/clang-php-checker
This will report such errors.
The mistakes you've mentioned are handled correctly in ther, but better
double check the type list in
https://github.com/johannes/clang-php-checker/blob/master/PHPZPPChecker.cpp#L72
in case there have been other recent changes. I haven't followed the
development.
johannes
Attachment: [application/pgp-signature] This is a digitally signed message part signature.asc
Attachment: [application/pgp-signature] This is a digitally signed message part signature.asc