Re: Re: svn: /php/php-src/ branches/PHP_5_3/ext/date/php_date.c trunk/ext/date/php_date.c
| From: | Derick Rethans | Date: | Tue, 25 Jan 2011 15:04:22 +0000 |
| Subject: | Re: Re: svn: /php/php-src/ branches/PHP_5_3/ext/date/php_date.c trunk/ext/date/php_date.c | ||
| References: | 1 2 3 4 5 6 7 8 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-51353@lists.php.net to get a copy of this message | ||
On Tue, 25 Jan 2011, Pierre Joye wrote:
> Also doing so (testing a == NULL instead of !a) is a common and widely
> adopted practice
Really? I've never written code like that. It's a preference. What you
do in your maintained extensions is up to you, but don't override
other people's preferences in how they test for NULL pointers. It's
perfectly legal to do it like it was. I'll be changing it back.
cheers,
Derick