Cleanup: get rid of sculpt minmax and reuse last stroke function (code
authorAntony Riakiotakis <kalast@gmail.com>
Mon, 29 Dec 2014 10:01:10 +0000 (11:01 +0100)
committerAntony Riakiotakis <kalast@gmail.com>
Mon, 29 Dec 2014 10:01:10 +0000 (11:01 +0100)
did that anyway, just kept the result in an intermediate variable)

source/blender/blenkernel/BKE_paint.h
source/blender/editors/include/ED_sculpt.h
source/blender/editors/sculpt_paint/sculpt.c
source/blender/editors/space_view3d/view3d_edit.c

index 8cf6fdcfeead021709e3eaf83918972e93a4f184..ff1305b918261019da09e49b7e1bafde34870083 100644 (file)
@@ -194,7 +194,6 @@ typedef struct SculptSession {
 
        /* last paint/sculpt stroke location */
        bool last_stroke_valid;
-       float last_stroke[3];
 
        float average_stroke_accum[3];
        int average_stroke_counter;
index 85ff9b5d2460a2dc66653b0e7b12c3096ebc1811..60c7dc8b8c7f24fa65b1b5d459391a99121cf247 100644 (file)
@@ -42,7 +42,6 @@ void ED_operatortypes_sculpt(void);
 void ED_sculpt_redraw_planes_get(float planes[4][4], struct ARegion *ar,
                                  struct RegionView3D *rv3d, struct Object *ob);
 void ED_sculpt_stroke_get_average(struct Object *ob, float stroke[3]);
-bool ED_sculpt_minmax(struct bContext *C, float min[3], float max[3]);
 int  ED_sculpt_mask_box_select(struct bContext *C, struct ViewContext *vc, const struct rcti *rect, bool select, bool extend);
 
 #endif /* __ED_SCULPT_H__ */
index 6df5c146b490ca8615c756e2e94c9009a0290c7b..50587c87bea5a75600b262612479b8f1a36d586b 100644 (file)
@@ -126,22 +126,6 @@ void ED_sculpt_stroke_get_average(Object *ob, float stroke[3])
        }
 }
 
-bool ED_sculpt_minmax(bContext *C, float min[3], float max[3])
-{
-       Object *ob = CTX_data_active_object(C);
-
-       if (ob && ob->sculpt && ob->sculpt->last_stroke_valid) {
-               copy_v3_v3(min, ob->sculpt->last_stroke);
-               copy_v3_v3(max, ob->sculpt->last_stroke);
-
-               return 1;
-       }
-       else {
-               return 0;
-       }
-}
-
-
 /* Check if there are any active modifiers in stack (used for flushing updates at enter/exit sculpt mode) */
 static bool sculpt_has_active_modifiers(Scene *scene, Object *ob)
 {
@@ -4366,7 +4350,6 @@ static void sculpt_stroke_done(const bContext *C, struct PaintStroke *UNUSED(str
 
                /* update last stroke position */
                ob->sculpt->last_stroke_valid = 1;
-               ED_sculpt_stroke_get_average(ob, ob->sculpt->last_stroke);
 
                sculpt_cache_free(ss->cache);
                ss->cache = NULL;
index 8ee6c7d8df459740ac5a430a95ca49ab9325e84d..2f4502eddcd959f097253c134a05dbb3b1997981 100644 (file)
@@ -2976,7 +2976,9 @@ static int viewselected_exec(bContext *C, wmOperator *op)
                ok = PE_minmax(scene, min, max);
        }
        else if (ob && (ob->mode & OB_MODE_SCULPT)) {
-               ok = ED_sculpt_minmax(C, min, max);
+               ED_sculpt_stroke_get_average(ob, min);
+               copy_v3_v3(max, min);
+               ok = true;
                ok_dist = 0; /* don't zoom */
        }
        else {