Re: CVS update: php3/functions
| From: | Rasmus Lerdorf | Date: | Thu, 01 Jan 1970 00:00:00 +0000 |
| Subject: | Re: CVS update: php3/functions | ||
| References: | 1 | Groups: | php.dev |
| Request: | Send a blank email to php-dev+get-2945@lists.php.net to get a copy of this message | ||
> -pedantic -Wall -Wmissing-prototypes -Wmissing-declarations
>
> If one ignores warnings about inline functions, PHP (almost) compiles quietly.
We have had this discussion before. The agreed upon flags are in
--enable-debug and they are like yours except without -pedantic. I think
adding this one is going too far. Wild casting like what you just added
does not help anybody. Anything that generates a warning under:
-Wall -Wmissing-prototypes -Wmissing-declarations
should be fixed. The kind of casting you added is more likely to hide
bugs than fix them. Also, as per the gcc man page regarding the -pedantic
option:
"There is no reason to use this option; it exists only to satisfy
pedants."
Unless you can make a good case for these changes I think you should
reverse them.
-Rasmus
--
PHP Development Mailing List http://www.php.net/
To unsubscribe send an empty message to php-dev-unsubscribe@lists.php.net
For help: php-dev-help@lists.php.net