[Bf-blender-cvs] [d98b38f97aa] master: USD Exporter: removed exclusion of 'Holdout' collections
Sybren A. Stüvel
noreply at git.blender.org
Thu Jan 9 15:52:58 CET 2020
Commit: d98b38f97aa07edfbaf81ab07523a1abef585ba1
Author: Sybren A. Stüvel
Date: Thu Jan 9 10:51:49 2020 +0100
Branches: master
https://developer.blender.org/rBd98b38f97aa07edfbaf81ab07523a1abef585ba1
USD Exporter: removed exclusion of 'Holdout' collections
This is in response to @brecht's remark in rBec62413f803e, where he
states that the approach was problematically interpreting the holdout
setting in a different way than what it was designed to do.
If we later want to add back a different "never include this in exports"
criterion, it can be easily done in
`AbstractHierarchyIterator::mark_as_weak_export()`. If such a criterion
should be file-format-specific, it can be done by overriding that
function in the file-format-specific subclass.
===================================================================
M source/blender/usd/intern/abstract_hierarchy_iterator.cc
===================================================================
diff --git a/source/blender/usd/intern/abstract_hierarchy_iterator.cc b/source/blender/usd/intern/abstract_hierarchy_iterator.cc
index 3ad2b2ce5d8..957db1a25ef 100644
--- a/source/blender/usd/intern/abstract_hierarchy_iterator.cc
+++ b/source/blender/usd/intern/abstract_hierarchy_iterator.cc
@@ -174,11 +174,6 @@ void AbstractHierarchyIterator::export_graph_construct()
object,
DEG_ITER_OBJECT_FLAG_LINKED_DIRECTLY |
DEG_ITER_OBJECT_FLAG_LINKED_VIA_SET) {
- if (object->base_flag & BASE_HOLDOUT) {
- visit_object(object, object->parent, true);
- continue;
- }
-
// Non-instanced objects always have their object-parent as export-parent.
const bool weak_export = mark_as_weak_export(object);
visit_object(object, object->parent, weak_export);
More information about the Bf-blender-cvs
mailing list