Commit: 2abb156b9f64dd0ec7d51aba8eac688fe22621ba
Author: Brecht Van Lommel
Date:   Fri Jun 8 18:43:05 2018 +0200
Branches: blender2.8
https://developer.blender.org/rB2abb156b9f64dd0ec7d51aba8eac688fe22621ba

Cleanup: remove object.is_visible.

Depsgraph already iterates over visible objects, and since this was only
valid for objects evaluated with the depsgraph it was confusing.

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

M       intern/cycles/blender/blender_object.cpp
M       source/blender/makesrna/intern/rna_object.c

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

diff --git a/intern/cycles/blender/blender_object.cpp 
b/intern/cycles/blender/blender_object.cpp
index 8ed3eafb488..dbeab2fdf61 100644
--- a/intern/cycles/blender/blender_object.cpp
+++ b/intern/cycles/blender/blender_object.cpp
@@ -578,9 +578,6 @@ void BlenderSync::sync_objects(BL::Depsgraph& b_depsgraph, 
float motion_time)
        {
                BL::DepsgraphObjectInstance b_instance = *b_instance_iter;
                BL::Object b_ob = b_instance.object();
-               if(!b_ob.is_visible()) {
-                       continue;
-               }
 
                progress.set_sync_status("Synchronizing object", b_ob.name());
 
diff --git a/source/blender/makesrna/intern/rna_object.c 
b/source/blender/makesrna/intern/rna_object.c
index 22a32586ff1..28e13790d5c 100644
--- a/source/blender/makesrna/intern/rna_object.c
+++ b/source/blender/makesrna/intern/rna_object.c
@@ -224,18 +224,6 @@ static void rna_Object_hide_update(Main *bmain, Scene 
*UNUSED(scene), PointerRNA
        DEG_id_type_tag(bmain, ID_OB);
 }
 
-static int rna_Object_is_visible_get(PointerRNA *ptr)
-{
-       Object *ob = ptr->id.data;
-       /* The duplicators final visibility is not evaluated by depsgraph, so 
it's
-        * in ob->base_flag & VISIBLED. Instead we need to take into account 
whether
-        * we are rendering or not, and the ob->duplicator_visibility_flag.
-        * However for this assessor we don't know if we are rendering, so we 
just
-        * ignore the duplicator visibility
-        */
-       return BKE_object_is_visible(ob, 
OB_VISIBILITY_CHECK_UNKNOWN_RENDER_MODE);
-}
-
 static void rna_Object_matrix_local_get(PointerRNA *ptr, float values[16])
 {
        Object *ob = ptr->id.data;
@@ -2302,11 +2290,6 @@ static void rna_def_object(BlenderRNA *brna)
        RNA_def_property_boolean_sdna(prop, NULL, "duplicator_visibility_flag", 
OB_DUPLI_FLAG_VIEWPORT);
        RNA_def_property_ui_text(prop, "Display Duplicator", "Make duplicator 
visible in the viewport");
 
-       prop = RNA_def_property(srna, "is_visible", PROP_BOOLEAN, PROP_NONE);
-       RNA_def_property_boolean_funcs(prop, "rna_Object_is_visible_get", NULL);
-       RNA_def_property_ui_text(prop, "Visible", "Visible to camera rays, set 
only on objects evaluated by depsgraph");
-       RNA_def_property_clear_flag(prop, PROP_EDITABLE);
-
        /* anim */
        rna_def_animdata_common(srna);

_______________________________________________
Bf-blender-cvs mailing list
[email protected]
https://lists.blender.org/mailman/listinfo/bf-blender-cvs

Reply via email to