From: Dirk R. <dre...@ia...> - 2004-04-01 18:00:18
|
Hi folks, On Mon, 2004-03-29 at 12:11, Chad Austin wrote: > Hm, it seems kind of silly to have to repeatedly call MergeGraphOp. This also > means that you can't just merge all geometry by adding a single MergeGraphOp to > the loader's GraphOpSeq. to be honest I'm a little surprised that it makes a difference. That sounds like there's a bug in the MergeGraphOp, as I can't see how it could miss something that it finds on the second run. > Would anyone be opposed to adding: > > size_t c; > do { > c = countNodes(node); > traverse(node); > } while (c > countNodes(node)); > > to the MergeGraphOp traverse? I'm okay with adding it as a workaround, but please add a comment about it being a bug in waiting. Thanks Dirk |