Stefan,

Could you refactor your test that uses unsetProperty to do it a different
way?  Is there any other use-case for having an unsetProperty method?   I
took a quick glance at the test case and it seems it could be broken into
two test methods so you wouldn't need unsetProperty - although I might not
be seeing the whole picture.

Would it be ok to return back a copy of the properties hashtable rather than
the actual object?  Any objections?

Thanks,
    Erik





----- Original Message -----
From: "Stefan Bodewig" <[EMAIL PROTECTED]>
To: <[EMAIL PROTECTED]>
Sent: Tuesday, November 27, 2001 3:17 AM
Subject: Re: <available> / <condition> breaking immutability


> On Mon, 26 Nov 2001, Erik Hatcher <[EMAIL PROTECTED]>
> wrote:
>
> > We should deprecate Project.unsetProperty also, right?
>
> I added it a few weeks ago and use it from inside a testcase AFAIR.
>
> All changes will be moot as long as we keep Project.getProperties
> around - or change it to return a copy the the "real" Hashtable.
>
> Stefan
>
> --
> To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
>
>


--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to