On Oct 15, 2010, at 12:39 PM, Kuba Ober wrote:

> In my porting to C++ data structures, I've made xobjs.pagelist a 
> QList<Pagedata>.
> QList's internal implementation is an array of pointers to elements, so it 
> seems a perfect fit here.
> 
> I ran into a few places in the code where you expect NULL pointers in that 
> list,
> and a few other places that pretend a NULL pointer won't ever happen in the 
> pagelist.
> 
> The code in changepage() and pre_initialize() seems to never insert any
> NULL pointers into the pagelist. Are there other places that I missed that 
> can put
> NULL pointers into the pagelist, or is it not a valid concern anymore?

Sorry, never mind, I got confused by a local variable called pagelist in 
savefile(short).

Cheers, Kuba
_______________________________________________
Xcircuit-dev mailing list
[email protected]
http://www.opencircuitdesign.com/mailman/listinfo/xcircuit-dev

Reply via email to