On Wed, 2004-09-29 at 15:25, Lars Gullik Bjønnes wrote: > Two newlines between functions please. That one doesn't seem to be in the Rules document, maybe you should add it.
> drop the if. (and change to use a smart pointer) Confession time: I have no idea about smart pointers. I've looked at the boost docs, and I don't Get It. I see what they're for and why they're useful, but I don't understand how to use them. It would help if you could point out exactly how to use the boost pointers in this particular situation (since I'm bound to run into very similar circumstances in other cases). > Do you really need it to be a pointer at all? I need to construct it with arguments. Is it possible to do that without instantiating it on the heap? Recall that I'm not very C++-experienced. > Why not call manage on it? Because manage is for widgets in containers: it frees the object when the container that it's packed into is destroyed. > space after // Again, I can't see this in Rules. John