[interfacekit] Re: BDeskbar weirdness

> 
> >GetItemInfo() points *found_name to the name of the item identified 
> > by 
> >for_id ...
> 
> >The caller is responsible for freeing found_name
> 
> >RETURN CODES:
> >...
> >B_BAD_VALUE (GetItemInfo()) *found_name is NULL
> 
> [...]
> >Any comments or concerns?  Anyone as perplexed as me about what Be 
> > was 
> >thinking when implementing this?  Can anyone explain why (*
> > found_name !
> >= NULL) is important?  Thanks.
> 
> Yes, it's pretty strange and does not really make sense.
> I have checked the Deskbar sources and it does nothing about this, 
> so, in the 
> original implementation, it's the BDeskbar class which performs the 
> (useless) 
> check and also makes a copy of the name.

Mmh, maybe it's a typo of the BDeskbar implementor and should be 
`found_name != NULL'. Anyway, there are a lot of bugs/weirdnesses in 
the API, so don't worry. ;-) For the Storage Kit we record the R5 
oddities we encounter and comment problems with our implementation in 
docs/develop/storage/Annotations. Maybe it's a good idea to do that for 
the other kits as well.

CU, Ingo

Other related posts: