[dev] Re: [cvs] commit: imp/lib/MIME/Viewer tnef.php

[Chuck Hagenbuch]

Quoting Michael M Slusarz <slusarz@bigworm.colorado.edu>:

> BTW, just a style thing, but I have noticed that (generally) isset() is 
> used in the codebase to check for the existence of an array key. I read in
> an article somewhere, and have been trying to do this myself, that 
> array_key_exists() is the preferable means of doing this.  (It is true
> that array_key_exists() has only been around since PHP 4.1.0).  If I 
> remember correctly, the two reasons to use it instead are 1) it is more a 
> more clear description and 2) isset does not work if the value is NULL.  
> Just wanted to throw this out there.

Sounds reasonable to me - if you see instances of this, feel free to change 
them in HEAD (not RELENG, though).

-chuck

--
Charles Hagenbuch, <chuck@horde.org>
"What was and what may be, lie, like children whose faces we cannot see, in 
the arms of silence. All we ever have is here, now." - Ursula K. Le Guin