diff options
author | Campbell Barton <ideasman42@gmail.com> | 2021-09-14 06:27:15 +0300 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2021-09-14 06:27:15 +0300 |
commit | 30845b5c8e042f42331b0c2a8a57ad05837b114e (patch) | |
tree | 9f50dfe1fb77c9a8f78f79a295333d668fd719c8 | |
parent | 917a972b56af103aee406dfffe1f42745b5ad360 (diff) |
Doc: expand on comment for why bound-box access could cause issues
-rw-r--r-- | source/blender/blenkernel/intern/constraint.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/source/blender/blenkernel/intern/constraint.c b/source/blender/blenkernel/intern/constraint.c index 72f14d94833..b9b15eba6a4 100644 --- a/source/blender/blenkernel/intern/constraint.c +++ b/source/blender/blenkernel/intern/constraint.c @@ -3900,7 +3900,11 @@ static void clampto_evaluate(bConstraint *con, bConstraintOb *cob, ListBase *tar unit_m4(targetMatrix); INIT_MINMAX(curveMin, curveMax); - /* XXX(campbell): don't think this is good calling this here. */ + /* XXX(@campbellbarton): don't think this is good calling this here because + * the other object's data is lazily initializing bounding-box information. + * This could cause issues when evaluating from a thread. + * If the depsgraph ensures the bound-box is always available, a code-path could + * be used that doesn't lazy initialize to avoid thread safety issues in the future. */ BKE_object_minmax(ct->tar, curveMin, curveMax, true); /* get targetmatrix */ |