On Aug 10, 2012, at 11:45 AM, Tedd Sperling <tedd@xxxxxxxxxxxx> wrote: > On Aug 9, 2012, at 5:16 PM, Jim Lucas <lists@xxxxxxxxx> wrote: >> You are relying on PHP's loose typing. This is a poor check. >> >> session_id() returns a string, not boolean. >> >> You should do this instead. >> >> if ( session_id() === '' ) >> .... >> >> >> -- >> Jim Lucas > > Thanks Jim -- you're right. > > What about? > > if (!defined(SID)) > { > session_start(); > } Before you answer, the "(!defined(SID))" is over 50 times slower than "( session_id() === '' )" Your way is better. Cheers, tedd _____________________ tedd@xxxxxxxxxxxx http://sperling.com -- PHP General Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php