Bugfix #21757: Crash when setting up cyclic tracking dependencies (with old tracking)
authorJoshua Leung <aligorith@gmail.com>
Fri, 26 Mar 2010 01:11:03 +0000 (01:11 +0000)
committerJoshua Leung <aligorith@gmail.com>
Fri, 26 Mar 2010 01:11:03 +0000 (01:11 +0000)
commitfedabce47c193926eb0c845a305a62bc73d8d6ce
treecb7cc9d67884e51af0073cdcb45f88ce1020863f
parent7a56ca7d6f4655a1c673b38f56b36953fcac4b15
Bugfix #21757: Crash when setting up cyclic tracking dependencies (with old tracking)

Note that users should not be doing this anyway (and to some degree, I wish that they have to learn this the hard way - i.e. a crash as was before) since it is always bound to cause troubles of various sorts.

Having said this, the old tracking code was previously crashing if this sort of setup was created since a stack overflow would happen while bouncing between each object being recursively recalculated. I've fixed this by commenting out that recursive recalculation (solving the cyclic problems for n >= 2, while n=1 should still be fine without this pre-depsgraph hack), and also removing such cyclic dependencies in the n=2 case.

(PS: Perhaps this is just a good opportunity to just remove this old feature instead ;)
source/blender/blenkernel/intern/object.c