Re: Two more phd patches
| From: | Hannes Magnusson | Date: | Fri, 03 Oct 2008 07:25:24 +0000 |
| Subject: | Re: Two more phd patches | ||
| References: | 1 | Groups: | php.doc |
| Request: | Send a blank email to phpdoc+get-969379465@lists.php.net to get a copy of this message | ||
On Fri, Oct 3, 2008 at 09:17, Christian Weiske <cweiske@cweiske.de> wrote:
> Hello,
>
>
> Attached are two phd patches that
> a) only fetch pear css files when they don't exist yet
> b) catch when getChildren() for a non-existing id is called. (happens in
> broken xml)
>
All these patches look fine - although I'd recommend
isset($array["key"]) over array_key_exists("key", $array) since
isset() is heckofalot faster (and it really matters in these functions
as they are executed at least 15.000 times..).
-Hannes
p.s. You should have PhD karma as of now, feel free to commit yourself :)