On Tue, Jan 25, 2011 at 4:04 PM, Derick Rethans <der...@php.net> wrote: > 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.
That's not really a surprise. > 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. You don't want one to commit in "your" extension? That's fine with me, but then keep it in pecl. Now, as this extension is in the core and had a couple of issues that need to be fixed and without affecting the behavior, then I will fix them. Features addition or major changes will obviously need review, as usual. Thanks for wasting my time with stupid things like that, Have a nice day, Cheers, -- Pierre @pierrejoye | http://blog.thepimp.net | http://www.libgd.org -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
