Re: pfNodeTravMask calls pfGetGSetBBox???

New Message Reply Date view Thread view Subject view Author view

Jim Helman (jimh++at++surreal)
Fri, 12 May 95 13:51:00 -0700

pfGetGSetBBox(..., NULL) is supposed to cause a recomputation of the
pfGeoSet bounding box, instead it core dumps. This is is a bug,
which I've just fixed. So no problem in 2.0.

As for why it was invoked: pfGSetISectMask was doing a clean of the
pfGeoSet's dirty attributes. In 1.2, when you change a pfGeoSet
attribute, the bounding box is marked dirty, with the updating done
at will thereafter. We can't recompute the bounding box whenever an
attribute changes, because we don't know if the data are valid yet
In the absence of a pfOKThisGSetIsDone() API, we implicitly make the
assumption that the pfGeoSet is self-consistent (i.e. either empty
or fully set up) when something other than an attribute set/get is
done. We could defer the "clean" to the first cull or isect
traversal, but this would require locks and would add non-
determinism, since the first time the object is "touched" would be
considerably more expensive.

rgds,

-jim helman

jimh++at++surreal.asd.sgi.com
415/390-1151

New Message Reply Date view Thread view Subject view Author view

This archive was generated by hypermail 2.0b2 on Mon Aug 10 1998 - 17:51:29 PDT

This message has been cleansed for anti-spam protection. Replace '++at++' in any mail addresses with the '@' symbol.