Re: cvs: php-src(PHP_5_1) /ext/xmlreader php_xmlreader.c

From: Date: Thu, 30 Mar 2006 21:02:31 +0000
Subject: Re: cvs: php-src(PHP_5_1) /ext/xmlreader php_xmlreader.c
References: 1 2 3 4 5  Groups: php.cvs 
Request: Send a blank email to php-cvs+get-37602@lists.php.net to get a copy of this message
Hello Hannes, Thursday, March 30, 2006, 9:26:27 PM, you wrote: > On 3/30/06, Marcus Boerger <helly@php.net> wrote: >> Hello Pierre, >> >> this is actually a bug fix not a feature change or whatever. >> if you do '$reader->getAttribute($name) you right now have no way of >> figuring out whether the attribute is empty or does not exist. I >> found that out while doing my slides on xmlReader over the weekend. >> We discussed it here in Montreal and all agreed that NULL is the best >> option since in contrast to false it also allows to use isset() and >> empty() on the return value. > Returning "" is however compatible with DOMElement->getAttribute()... If that is the case then it is an error there, too. I must admit i never use DOM so i don't know and don't care either. Best regards, Marcus

« previous php.cvs (#37602) next »