may I suggest that you add the same check also to that method :
protected void removeAttributeInternal(String name, boolean notify) {
// Remove this attribute from our collection
Object value = attributes.remove(name);
?
Thanx
Leon.
On Mon, Jun 16, 2008 at 9:25 AM, Mark Thomas <[EMAIL PROTECTED]> wrote:
>
> Leon Rosenberg wrote:
>>
>> P.S. probably it would be good if StandartSession would check for it
>> return name==null ? null : (attributes.get(name));
>> instead of
>> return (attributes.get(name));
>>
>> but they probably won't do it anyway:-)
>
> Oh yea of little faith.
>
> http://svn.apache.org/viewvc?rev=667604&view=rev
> and proposed for 5.5.x and 6.0.x
>
> Of course it needs 3 +1s to get in to the release branches...
>
> Mark
>
>
> ---------------------------------------------------------------------
> To start a new topic, e-mail: [email protected]
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>
---------------------------------------------------------------------
To start a new topic, e-mail: [email protected]
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
