[Bf-blender-cvs] [c80411ce3b1] blender-v3.1-release: Despgraph: Remove asserts from object modifier backup

Sergey Sharybin noreply at git.blender.org
Thu Feb 3 10:02:26 CET 2022


Commit: c80411ce3b14a6b9d2181eb8c8c921932ed2709d
Author: Sergey Sharybin
Date:   Wed Feb 2 11:55:04 2022 +0100
Branches: blender-v3.1-release
https://developer.blender.org/rBc80411ce3b14a6b9d2181eb8c8c921932ed2709d

Despgraph: Remove asserts from object modifier backup

The modifiers are mapped between original and evaluated objects based on
their session IDs. The pointer to original modifier is no longer needed
for the backup: it remained from the initial implementation which was
rewritten at some point.

This is a preparation for removal of the pointer to original modifier.

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

M	source/blender/depsgraph/intern/eval/deg_eval_runtime_backup_object.cc

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

diff --git a/source/blender/depsgraph/intern/eval/deg_eval_runtime_backup_object.cc b/source/blender/depsgraph/intern/eval/deg_eval_runtime_backup_object.cc
index 1081528ece1..17369f723ec 100644
--- a/source/blender/depsgraph/intern/eval/deg_eval_runtime_backup_object.cc
+++ b/source/blender/depsgraph/intern/eval/deg_eval_runtime_backup_object.cc
@@ -71,7 +71,6 @@ void ObjectRuntimeBackup::backup_modifier_runtime_data(Object *object)
     const SessionUUID &session_uuid = modifier_data->session_uuid;
     BLI_assert(BLI_session_uuid_is_generated(&session_uuid));
 
-    BLI_assert(modifier_data->orig_modifier_data != nullptr);
     modifier_runtime_data.add(session_uuid, ModifierDataBackup(modifier_data));
     modifier_data->runtime = nullptr;
   }
@@ -150,8 +149,9 @@ void ObjectRuntimeBackup::restore_to_object(Object *object)
 void ObjectRuntimeBackup::restore_modifier_runtime_data(Object *object)
 {
   LISTBASE_FOREACH (ModifierData *, modifier_data, &object->modifiers) {
-    BLI_assert(modifier_data->orig_modifier_data != nullptr);
     const SessionUUID &session_uuid = modifier_data->session_uuid;
+    BLI_assert(BLI_session_uuid_is_generated(&session_uuid));
+
     optional<ModifierDataBackup> backup = modifier_runtime_data.pop_try(session_uuid);
     if (backup.has_value()) {
       modifier_data->runtime = backup->runtime;



More information about the Bf-blender-cvs mailing list