24 Apr
2004
24 Apr
'04
4:01 p.m.
Hi,
While doing some SPSelection OO-ification I have noticed that there seems to be some confusion in the code about whether desktop->selection is ever NULL in normal operation. Anyone got any clues?
Basically I think this dictates whether desktop->selection should be g_assert()ed non-NULL or assumed empty if it's NULL.
The same confusion arises in other places where a SPSelection* is held. Personally, I'd favour the convention that all SPSelection* in structs are assumed to be non-NULL.
Thanks
Carl