<html>
<head>
<style>
P
{
margin:0px;
padding:0px
}
body
{
FONT-SIZE: 10pt;
FONT-FAMILY:Tahoma
}
</style>
</head>
<body>Jonathan<BR>
<BR>
Hmmm, I think I understand your reasoning in that trapping for null may hide a bigger problem elsewhere. I am not too sure that there is enough justification in this case. It's no big issue as the message that spils out of PHP is merely a notification. Many people will have these turned off anyhow. <BR>
<BR>
The issue I have is that the library is now building in assumptions as to how the user of the library is going to use it. i.e. they are going to the native PHP session mechanism. I dont use this, or more correctly, the framework I am currently working in does not use this. It can, but it does not. There are a fair number of frameworks out there that use their own mechanisms. In my case, and many others therefore, this error will always be thrown.<BR>
<BR>
I would also suggest that the reason for keeping it there, i.e. it will highlight problems earlier is probably not that valid as it is only a notification and many PHP installations will have these switched off. If there is a problem, a real problem, it is more than likely going to be found closer to the coal face than by examining logs with notifications switched on.<BR>
<BR>
I hope you can appreciate these are merely points on a relatively insignificant and generally benign problem. As you are the author of the library, its your call. If you dont patch, then the likelihood is that I'll have to patch every release. I'll have to take a look at the next post of licencing to make sure how that effects me.<BR>
<BR>
<BR><br /><hr />With Live Spaces email straight to your blog. Upload jokes, photos and more. It's free! <a href='http://specials.uk.msn.com/spaces/default.aspx ' target='_new'>It's free!</a></body>
</html>