[Bf-blender-cvs] [79558a581de] master: CodeCleanup: Use eDrawType

Jeroen Bakker noreply at git.blender.org
Fri Mar 6 12:23:49 CET 2020


Commit: 79558a581de707413041ab6f6af94ac5309b1dba
Author: Jeroen Bakker
Date:   Fri Mar 6 12:21:39 2020 +0100
Branches: master
https://developer.blender.org/rB79558a581de707413041ab6f6af94ac5309b1dba

CodeCleanup: Use eDrawType

This patch will make use of the eDrawType where it was used as a
variable or parameter name. The eObjectDrawType was renamed to eDrawType
as it is also used by `View3DShading.type`.

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

M	source/blender/blenkernel/BKE_sequencer_offscreen.h
M	source/blender/draw/engines/select/select_draw_utils.c
M	source/blender/draw/intern/draw_manager.c
M	source/blender/editors/include/ED_view3d_offscreen.h
M	source/blender/editors/space_view3d/view3d_draw.c
M	source/blender/makesdna/DNA_object_enums.h

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

diff --git a/source/blender/blenkernel/BKE_sequencer_offscreen.h b/source/blender/blenkernel/BKE_sequencer_offscreen.h
index 72e3f246dcb..0385985b019 100644
--- a/source/blender/blenkernel/BKE_sequencer_offscreen.h
+++ b/source/blender/blenkernel/BKE_sequencer_offscreen.h
@@ -37,7 +37,7 @@ extern "C" {
 typedef struct ImBuf *(*SequencerDrawView)(struct Depsgraph *depsgraph,
                                            struct Scene *scene,
                                            struct View3DShading *shading_override,
-                                           eObjectDrawType drawtype,
+                                           eDrawType drawtype,
                                            struct Object *camera,
                                            int width,
                                            int height,
diff --git a/source/blender/draw/engines/select/select_draw_utils.c b/source/blender/draw/engines/select/select_draw_utils.c
index f1d008c29c7..32bacdfab78 100644
--- a/source/blender/draw/engines/select/select_draw_utils.c
+++ b/source/blender/draw/engines/select/select_draw_utils.c
@@ -83,7 +83,7 @@ short select_id_get_object_select_mode(Scene *scene, Object *ob)
   return r_select_mode;
 }
 
-static bool check_ob_drawface_dot(short select_mode, const View3D *v3d, char dt)
+static bool check_ob_drawface_dot(short select_mode, const View3D *v3d, eDrawType dt)
 {
   if (select_mode & SCE_SELECT_FACE) {
     if ((dt < OB_SOLID) || XRAY_FLAG_ENABLED(v3d)) {
diff --git a/source/blender/draw/intern/draw_manager.c b/source/blender/draw/intern/draw_manager.c
index 80001144cc0..cd9420feb77 100644
--- a/source/blender/draw/intern/draw_manager.c
+++ b/source/blender/draw/intern/draw_manager.c
@@ -1130,7 +1130,7 @@ static void drw_engines_enable_external(void)
 /* Gather all draw engines needed and store them in DST.enabled_engines
  * That also define the rendering order of engines */
 static void drw_engines_enable_from_engine(RenderEngineType *engine_type,
-                                           int drawtype,
+                                           eDrawType drawtype,
                                            bool use_xray)
 {
   switch (drawtype) {
@@ -1179,7 +1179,7 @@ static void drw_engines_enable(ViewLayer *UNUSED(view_layer),
                                bool gpencil_engine_needed)
 {
   View3D *v3d = DST.draw_ctx.v3d;
-  const int drawtype = v3d->shading.type;
+  const eDrawType drawtype = v3d->shading.type;
   const bool use_xray = XRAY_ENABLED(v3d);
 
   drw_engines_enable_from_engine(engine_type, drawtype, use_xray);
diff --git a/source/blender/editors/include/ED_view3d_offscreen.h b/source/blender/editors/include/ED_view3d_offscreen.h
index 5a31516ec64..4c0d081bcd8 100644
--- a/source/blender/editors/include/ED_view3d_offscreen.h
+++ b/source/blender/editors/include/ED_view3d_offscreen.h
@@ -43,7 +43,7 @@ struct View3DShading;
 
 void ED_view3d_draw_offscreen(struct Depsgraph *depsgraph,
                               struct Scene *scene,
-                              eObjectDrawType drawtype,
+                              eDrawType drawtype,
                               struct View3D *v3d,
                               struct ARegion *ar,
                               int winx,
@@ -59,7 +59,7 @@ void ED_view3d_draw_offscreen(struct Depsgraph *depsgraph,
 
 struct ImBuf *ED_view3d_draw_offscreen_imbuf(struct Depsgraph *depsgraph,
                                              struct Scene *scene,
-                                             eObjectDrawType drawtype,
+                                             eDrawType drawtype,
                                              struct View3D *v3d,
                                              struct ARegion *ar,
                                              int sizex,
@@ -72,7 +72,7 @@ struct ImBuf *ED_view3d_draw_offscreen_imbuf(struct Depsgraph *depsgraph,
 struct ImBuf *ED_view3d_draw_offscreen_imbuf_simple(struct Depsgraph *depsgraph,
                                                     struct Scene *scene,
                                                     struct View3DShading *shading_override,
-                                                    eObjectDrawType drawtype,
+                                                    eDrawType drawtype,
                                                     struct Object *camera,
                                                     int width,
                                                     int height,
diff --git a/source/blender/editors/space_view3d/view3d_draw.c b/source/blender/editors/space_view3d/view3d_draw.c
index c58c4b33f05..101e04664ec 100644
--- a/source/blender/editors/space_view3d/view3d_draw.c
+++ b/source/blender/editors/space_view3d/view3d_draw.c
@@ -1631,7 +1631,7 @@ static void view3d_stereo3d_setup_offscreen(Depsgraph *depsgraph,
 
 void ED_view3d_draw_offscreen(Depsgraph *depsgraph,
                               Scene *scene,
-                              eObjectDrawType drawtype,
+                              eDrawType drawtype,
                               View3D *v3d,
                               ARegion *ar,
                               int winx,
@@ -1711,7 +1711,7 @@ void ED_view3d_draw_offscreen(Depsgraph *depsgraph,
  */
 ImBuf *ED_view3d_draw_offscreen_imbuf(Depsgraph *depsgraph,
                                       Scene *scene,
-                                      eObjectDrawType drawtype,
+                                      eDrawType drawtype,
                                       View3D *v3d,
                                       ARegion *ar,
                                       int sizex,
@@ -1857,7 +1857,7 @@ ImBuf *ED_view3d_draw_offscreen_imbuf(Depsgraph *depsgraph,
 ImBuf *ED_view3d_draw_offscreen_imbuf_simple(Depsgraph *depsgraph,
                                              Scene *scene,
                                              View3DShading *shading_override,
-                                             eObjectDrawType drawtype,
+                                             eDrawType drawtype,
                                              Object *camera,
                                              int width,
                                              int height,
diff --git a/source/blender/makesdna/DNA_object_enums.h b/source/blender/makesdna/DNA_object_enums.h
index 3ba26dc4bb5..c15f32564c4 100644
--- a/source/blender/makesdna/DNA_object_enums.h
+++ b/source/blender/makesdna/DNA_object_enums.h
@@ -39,15 +39,15 @@ typedef enum eObjectMode {
   OB_MODE_WEIGHT_GPENCIL = 1 << 10,
 } eObjectMode;
 
-/** #Object.dt */
-typedef enum eObjectDrawType {
+/** #Object.dt, #View3DShading.type */
+typedef enum eDrawType {
   OB_BOUNDBOX = 1,
   OB_WIRE = 2,
   OB_SOLID = 3,
   OB_MATERIAL = 4,
   OB_TEXTURE = 5,
   OB_RENDER = 6,
-} eObjectDrawType;
+} eDrawType;
 
 /** Any mode where the brush system is used. */
 #define OB_MODE_ALL_PAINT \



More information about the Bf-blender-cvs mailing list