Re: [cp-patches] FYI: Allow Security.setProperty("foo", null)

[Gary Benson]

Tom Tromey wrote:
> >>>>> "Gary" == Gary Benson <address@hidden> writes:
> 
> Gary> At the moment Security.setProperty() will not allow the
> Gary> setting of null property values.  Since Security.getProperty()
> Gary> returns null for unset properties this means that the
> Gary> following will fail:
> 
> When you sent this last week I replied to it... did you not see
> that?

No, I did not.  I certainly wouldn't have committed without replying.

I seem to have missed a bunch of mails lately.  I wonder if someone's
installed an overzealous spam filter upstream of me.

> In particular I wanted to know about Mauve tests and documenting the
> behavior of 'null'.

With the patch Security.setProperty("foo", null) will unset the
property.  Security.getProperty("foo") returns null for unset
properties.

I'll do a Mauve test now...

> Also your ChangeLog entry is not really formatted properly.  I
> wouldn't bother changing that at this point though, it is just a
> pedantic thing.

Ok, sorry.

Cheers,
Gary