[Bf-blender-cvs] [8c0698460be] master: Revert "Fix T92136: Leak accessing evaluated depsgraph data from Python"

[Campbell Barton]

Commit: 8c0698460be9e435560b0b00474900492a614ac0
Author: Campbell Barton
Date:   Wed Oct 13 21:29:27 2021 +1100
Branches: master
https://developer.blender.org/rB8c0698460be9e435560b0b00474900492a614ac0

Revert "Fix T92136: Leak accessing evaluated depsgraph data from Python"

This reverts commit 0558907ae674ebe81dc8910a6615fc32ce675d70.

Based on discussion with Sergey, having Python references to un-expanded
data should not happen - this change needs to be reconsidered.

===================================================================

M	source/blender/depsgraph/intern/eval/deg_eval_copy_on_write.cc
M	source/blender/depsgraph/intern/eval/deg_eval_runtime_backup.cc

===================================================================

diff --git a/source/blender/depsgraph/intern/eval/deg_eval_copy_on_write.cc b/source/blender/depsgraph/intern/eval/deg_eval_copy_on_write.cc
index 610e4860108..a844d23b558 100644
--- a/source/blender/depsgraph/intern/eval/deg_eval_copy_on_write.cc
+++ b/source/blender/depsgraph/intern/eval/deg_eval_copy_on_write.cc
@@ -853,10 +853,6 @@ ID *deg_expand_copy_on_write_datablock(const Depsgraph *depsgraph,
   if (!deg_copy_on_write_is_needed(id_orig)) {
     return id_cow;
   }
-  /* Avoid removing & re-creating the reference if it exists. */
-  void *py_instance = id_cow->py_instance;
-  id_cow->py_instance = nullptr;
-
   DEG_COW_PRINT(
       "Expanding datablock for %s: id_orig=%p id_cow=%p\n", id_orig->name, id_orig, id_cow);
   /* Sanity checks. */
@@ -929,7 +925,6 @@ ID *deg_expand_copy_on_write_datablock(const Depsgraph *depsgraph,
    * from above. */
   update_id_after_copy(depsgraph, id_node, id_orig, id_cow);
   id_cow->recalc = id_cow_recalc;
-  id_cow->py_instance = py_instance;
   return id_cow;
 }
 
@@ -1047,11 +1042,6 @@ void discard_edit_mode_pointers(ID *id_cow)
  * - Does not free data-block itself. */
 void deg_free_copy_on_write_datablock(ID *id_cow)
 {
-  /* There may be Python references to to shallow copies, see: T92136. */
-  if (id_cow->py_instance) {
-    BKE_libblock_free_data_py(id_cow);
-  }
-
   if (!check_datablock_expanded(id_cow)) {
     /* Actual content was never copied on top of CoW block, we have
      * nothing to free. */
@@ -1113,7 +1103,6 @@ void deg_tag_copy_on_write_id(ID *id_cow, const ID *id_orig)
   /* This ID is no longer localized, is a self-sustaining copy now. */
   id_cow->tag &= ~LIB_TAG_LOCALIZED;
   id_cow->orig_id = (ID *)id_orig;
-  BLI_assert(id_cow->py_instance == nullptr);
 }
 
 bool deg_copy_on_write_is_expanded(const ID *id_cow)
diff --git a/source/blender/depsgraph/intern/eval/deg_eval_runtime_backup.cc b/source/blender/depsgraph/intern/eval/deg_eval_runtime_backup.cc
index c6e3532aca9..7893e8c64c1 100644
--- a/source/blender/depsgraph/intern/eval/deg_eval_runtime_backup.cc
+++ b/source/blender/depsgraph/intern/eval/deg_eval_runtime_backup.cc
@@ -52,9 +52,7 @@ void RuntimeBackup::init_from_id(ID *id)
   }
   have_backup = true;
 
-  /* Clear, so freeing the expanded data doesn't remove this Python reference. */
   id_data.py_instance = id->py_instance;
-  id->py_instance = nullptr;
 
   animation_backup.init_from_id(id);