[dev] Re: [cvs] commit: horde/lib String.php

[Chuck Hagenbuch]

Quoting Jan Schneider <jan at horde.org>:

> Please revert this.
> I recognized during a performance audit that this function was a real
> performance hit. Chuck and me also tested the performance of
> array_key_exist() vs. isset() and detected that isset() is around twice
> as fast. Unless there is really a reason to use array_key_exists(), we
> should in the future use isset() again.

I agree with Jan, fwiw - I hadn't bothered to test this before, but you can
look at http://marina.horde.org/wip/isset.php and
http://marina.horde.org/wip/isset.phps (please copy and confirm locally).

-chuck

--
Charles Hagenbuch, <chuck at horde.org>
must ... find ... acorns ... *thud*