Commit: 05111d79d098376ce094058fbf2178e4ea418c38
Author: Brecht Van Lommel
Date:   Mon Jun 11 11:48:16 2018 +0200
Branches: blender2.8
https://developer.blender.org/rB05111d79d098376ce094058fbf2178e4ea418c38

Cleanup: remove unused DAG_EVAL_PREVIEW mode.

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

M       source/blender/blenkernel/intern/object_dupli.c
M       source/blender/depsgraph/DEG_depsgraph.h
M       source/blender/depsgraph/intern/depsgraph_query_iter.cc
M       source/blender/makesdna/DNA_scene_types.h
M       source/blender/makesrna/intern/rna_depsgraph.c
M       source/blender/render/intern/source/pipeline.c

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

diff --git a/source/blender/blenkernel/intern/object_dupli.c 
b/source/blender/blenkernel/intern/object_dupli.c
index 2d1c03cf883..e9b1e79dc67 100644
--- a/source/blender/blenkernel/intern/object_dupli.c
+++ b/source/blender/blenkernel/intern/object_dupli.c
@@ -498,7 +498,7 @@ static void make_duplis_verts(const DupliContext *ctx)
 {
        Scene *scene = ctx->scene;
        Object *parent = ctx->object;
-       bool use_texcoords = ELEM(DEG_get_mode(ctx->depsgraph), 
DAG_EVAL_RENDER, DAG_EVAL_PREVIEW);
+       bool use_texcoords = (DEG_get_mode(ctx->depsgraph) == DAG_EVAL_RENDER);
        VertexDupliData vdd;
 
        vdd.ctx = ctx;
@@ -702,7 +702,7 @@ static void make_child_duplis_faces(const DupliContext 
*ctx, void *userdata, Obj
        float (*orco)[3] = fdd->orco;
        MLoopUV *mloopuv = fdd->mloopuv;
        int a, totface = fdd->totface;
-       bool use_texcoords = ELEM(DEG_get_mode(ctx->depsgraph), 
DAG_EVAL_RENDER, DAG_EVAL_PREVIEW);
+       bool use_texcoords = (DEG_get_mode(ctx->depsgraph) == DAG_EVAL_RENDER);
        float child_imat[4][4];
        DupliObject *dob;
 
@@ -768,7 +768,7 @@ static void make_duplis_faces(const DupliContext *ctx)
 {
        Scene *scene = ctx->scene;
        Object *parent = ctx->object;
-       bool use_texcoords = ELEM(DEG_get_mode(ctx->depsgraph), 
DAG_EVAL_RENDER, DAG_EVAL_PREVIEW);
+       bool use_texcoords = (DEG_get_mode(ctx->depsgraph) == DAG_EVAL_RENDER);
        FaceDupliData fdd;
 
        fdd.use_scale = ((parent->transflag & OB_DUPLIFACES_SCALE) != 0);
@@ -821,7 +821,7 @@ static void make_duplis_particle_system(const DupliContext 
*ctx, ParticleSystem
        Scene *scene = ctx->scene;
        Object *par = ctx->object;
        bool for_render = DEG_get_mode(ctx->depsgraph) == DAG_EVAL_RENDER;
-       bool use_texcoords = ELEM(DEG_get_mode(ctx->depsgraph), 
DAG_EVAL_RENDER, DAG_EVAL_PREVIEW);
+       bool use_texcoords = for_render;
 
        Object *ob = NULL, **oblist = NULL, obcopy, *obcopylist = NULL;
        DupliObject *dob;
diff --git a/source/blender/depsgraph/DEG_depsgraph.h 
b/source/blender/depsgraph/DEG_depsgraph.h
index 1dd2e1e587a..0174b002292 100644
--- a/source/blender/depsgraph/DEG_depsgraph.h
+++ b/source/blender/depsgraph/DEG_depsgraph.h
@@ -70,8 +70,7 @@ struct ViewLayer;
 
 typedef enum eEvaluationMode {
        DAG_EVAL_VIEWPORT       = 0,    /* evaluate for OpenGL viewport */
-       DAG_EVAL_PREVIEW        = 1,    /* evaluate for render with preview 
settings */
-       DAG_EVAL_RENDER         = 2,    /* evaluate for render purposes */
+       DAG_EVAL_RENDER         = 1,    /* evaluate for render purposes */
 } eEvaluationMode;
 
 /* DagNode->eval_flags */
diff --git a/source/blender/depsgraph/intern/depsgraph_query_iter.cc 
b/source/blender/depsgraph/intern/depsgraph_query_iter.cc
index 97a28038b7b..5dca0e83124 100644
--- a/source/blender/depsgraph/intern/depsgraph_query_iter.cc
+++ b/source/blender/depsgraph/intern/depsgraph_query_iter.cc
@@ -212,7 +212,6 @@ void DEG_iterator_objects_begin(BLI_Iterator *iter, 
DEGObjectIterData *data)
        data->id_node_index = 0;
        data->num_id_nodes = num_id_nodes;
        eEvaluationMode eval_mode = DEG_get_mode(depsgraph);
-       /* Viewport rendered mode is DAG_EVAL_PREVIEW but still treated as 
viewport. */
        data->visibility_check = (eval_mode == DAG_EVAL_RENDER)
                                 ? OB_VISIBILITY_CHECK_FOR_RENDER
                                 : OB_VISIBILITY_CHECK_FOR_VIEWPORT;
diff --git a/source/blender/makesdna/DNA_scene_types.h 
b/source/blender/makesdna/DNA_scene_types.h
index ea6a34df3f5..0c2169072d3 100644
--- a/source/blender/makesdna/DNA_scene_types.h
+++ b/source/blender/makesdna/DNA_scene_types.h
@@ -1623,7 +1623,7 @@ enum {
 /* #define R_DEPRECATED                0x10000 */
 /* #define R_RECURS_PROTECTION 0x20000 */
 #define R_TEXNODE_PREVIEW      0x40000
-#define R_VIEWPORT_PREVIEW     0x80000
+/* #define R_VIEWPORT_PREVIEW  0x80000 */
 #define R_EXR_CACHE_FILE       0x100000
 #define R_MULTIVIEW                    0x200000
 
diff --git a/source/blender/makesrna/intern/rna_depsgraph.c 
b/source/blender/makesrna/intern/rna_depsgraph.c
index f904d4c06b1..3e85e225d27 100644
--- a/source/blender/makesrna/intern/rna_depsgraph.c
+++ b/source/blender/makesrna/intern/rna_depsgraph.c
@@ -474,7 +474,6 @@ static void rna_def_depsgraph(BlenderRNA *brna)
 
        static EnumPropertyItem enum_depsgraph_mode_items[] = {
                {DAG_EVAL_VIEWPORT, "VIEWPORT", 0, "Viewport", "Viewport 
non-rendered mode"},
-               {DAG_EVAL_PREVIEW, "PREVIEW", 0, "Preview", "Viewport rendered 
draw mode"},
                {DAG_EVAL_RENDER, "RENDER", 0, "Render", "Render"},
                {0, NULL, 0, NULL, NULL}
        };
diff --git a/source/blender/render/intern/source/pipeline.c 
b/source/blender/render/intern/source/pipeline.c
index 8493913bd29..54fb4251a3b 100644
--- a/source/blender/render/intern/source/pipeline.c
+++ b/source/blender/render/intern/source/pipeline.c
@@ -768,7 +768,7 @@ void RE_InitState(Render *re, Render *source, RenderData 
*rd,
        /* if preview render, we try to keep old result */
        BLI_rw_mutex_lock(&re->resultmutex, THREAD_LOCK_WRITE);
 
-       if (re->r.scemode & (R_BUTS_PREVIEW|R_VIEWPORT_PREVIEW)) {
+       if (re->r.scemode & R_BUTS_PREVIEW) {
                if (had_freestyle || (re->r.mode & R_EDGE_FRS)) {
                        /* freestyle manipulates render layers so always have 
to free */
                        render_result_free(re->result);
@@ -806,10 +806,6 @@ void RE_InitState(Render *re, Render *source, RenderData 
*rd,
                render_result_view_new(re->result, "");
        }
 
-       eEvaluationMode mode = (re->r.scemode & R_VIEWPORT_PREVIEW) ? 
DAG_EVAL_PREVIEW : DAG_EVAL_RENDER;
-       /* This mode should have been set in the Depsgraph immediately when it 
was created. */
-       (void)mode;
-
        /* ensure renderdatabase can use part settings correct */
        RE_parts_clamp(re);

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

Reply via email to