Re: Bug in setPersistentDomain?

[Andreas Heppel]

On 2002-11-11 18:46:14 +0000 Richard Frith-Macdonald <richard@brainstorm.co.uk> 
wrote:

> On Monday, November 11, 2002, at 01:40  pm, Andreas Heppel wrote:
>
> > Hi there,
> > I just checked some methods for NSUserDomain and saw that
> > - (void) setPersistentDomain:forName: checks the variable _tempDomains whether 
> > the requested domainname already exists. Shouldn't this check be made over 
> > _persDomains instead? If I get it right _tempDomains holds the volatile ones.
>
> The check shouldn't be there at all (I removed it) and the one in 
> setVolatileDomain:forName: was wrong ... I corrected it.
OK. But iirc Apple's Foundation manual says that both methods should raise 
exceptions if a persistent/volatile domain with the given name already exists, 
while the comment in gnustep-base reads 'Replaces the persistent...'. I 
actually prefer the latter (and now impelmented :-) version of GNUstep since it 
makes more sense to me. But wouldn't that break the *step specs?

Andreas

--
Andreas Heppel

Mail: aheppel at web dot de
Home: http://www.andreasheppel.de

Check out GSburn.app - the CD burning frontend for GNUstep
http://gsburn.sourceforge.net