utility functions for getting/setting rectangles for operators.
authorCampbell Barton <ideasman42@gmail.com>
Wed, 8 Aug 2012 20:38:55 +0000 (20:38 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Wed, 8 Aug 2012 20:38:55 +0000 (20:38 +0000)
23 files changed:
source/blender/editors/animation/anim_channels_edit.c
source/blender/editors/animation/anim_markers.c
source/blender/editors/animation/anim_ops.c
source/blender/editors/interface/view2d_ops.c
source/blender/editors/mask/mask_select.c
source/blender/editors/screen/screen_ops.c
source/blender/editors/sculpt_paint/paint_hide.c
source/blender/editors/space_action/action_select.c
source/blender/editors/space_clip/clip_graph_ops.c
source/blender/editors/space_clip/tracking_select.c
source/blender/editors/space_file/file_ops.c
source/blender/editors/space_graph/graph_select.c
source/blender/editors/space_info/info_report.c
source/blender/editors/space_nla/nla_select.c
source/blender/editors/space_node/node_select.c
source/blender/editors/space_outliner/outliner_select.c
source/blender/editors/space_sequencer/sequencer_select.c
source/blender/editors/space_view3d/view3d_edit.c
source/blender/editors/space_view3d/view3d_select.c
source/blender/editors/transform/transform.c
source/blender/editors/uvedit/uvedit_ops.c
source/blender/windowmanager/WM_api.h
source/blender/windowmanager/intern/wm_operators.c

index c10579d2d3086e77417c7a665ddf7074203f642d..d35f39a48e01fc9dba3c141f5868e20c959a840c 100644 (file)
@@ -1930,10 +1930,7 @@ static int animchannels_borderselect_exec(bContext *C, wmOperator *op)
                return OPERATOR_CANCELLED;
        
        /* get settings from operator */
-       rect.xmin = RNA_int_get(op->ptr, "xmin");
-       rect.ymin = RNA_int_get(op->ptr, "ymin");
-       rect.xmax = RNA_int_get(op->ptr, "xmax");
-       rect.ymax = RNA_int_get(op->ptr, "ymax");
+       WM_operator_properties_border_to_rcti(op, &rect);
        
        gesture_mode = RNA_int_get(op->ptr, "gesture_mode");
        extend = RNA_boolean_get(op->ptr, "extend");
index 96044e2408ef83b095a6c8cd4e17cb736424967b..a8daf852ddab96469e1af350775cf7495985baf5 100644 (file)
@@ -1146,14 +1146,13 @@ static int ed_marker_border_select_exec(bContext *C, wmOperator *op)
        TimeMarker *marker;
        float xminf, xmaxf, yminf, ymaxf;
        int gesture_mode = RNA_int_get(op->ptr, "gesture_mode");
-       int xmin = RNA_int_get(op->ptr, "xmin");
-       int xmax = RNA_int_get(op->ptr, "xmax");
-       int ymin = RNA_int_get(op->ptr, "ymin");
-       int ymax = RNA_int_get(op->ptr, "ymax");
        int extend = RNA_boolean_get(op->ptr, "extend");
+       rcti rect;
        
-       UI_view2d_region_to_view(v2d, xmin, ymin, &xminf, &yminf);      
-       UI_view2d_region_to_view(v2d, xmax, ymax, &xmaxf, &ymaxf);      
+       WM_operator_properties_border_to_rcti(op, &rect);
+
+       UI_view2d_region_to_view(v2d, rect.xmin, rect.ymin, &xminf, &yminf);
+       UI_view2d_region_to_view(v2d, rect.xmax, rect.ymax, &xmaxf, &ymaxf);
        
        if (markers == NULL)
                return 0;
index 1ab6247728a0235dcad6fd237d5401fb489fcf14..e24a4d49a0572451427b713d42e9a2093bee1969 100644 (file)
@@ -185,15 +185,14 @@ static int previewrange_define_exec(bContext *C, wmOperator *op)
        Scene *scene = CTX_data_scene(C);
        ARegion *ar = CTX_wm_region(C);
        float sfra, efra;
-       int xmin, xmax;
+       rcti rect;
        
        /* get min/max values from border select rect (already in region coordinates, not screen) */
-       xmin = RNA_int_get(op->ptr, "xmin");
-       xmax = RNA_int_get(op->ptr, "xmax");
+       WM_operator_properties_border_to_rcti(op, &rect);
        
        /* convert min/max values to frames (i.e. region to 'tot' rect) */
-       UI_view2d_region_to_view(&ar->v2d, xmin, 0, &sfra, NULL);
-       UI_view2d_region_to_view(&ar->v2d, xmax, 0, &efra, NULL);
+       UI_view2d_region_to_view(&ar->v2d, rect.xmin, 0, &sfra, NULL);
+       UI_view2d_region_to_view(&ar->v2d, rect.xmax, 0, &efra, NULL);
        
        /* set start/end frames for preview-range 
         *      - must clamp within allowable limits
@@ -232,12 +231,11 @@ static void ANIM_OT_previewrange_set(wmOperatorType *ot)
        ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
        
        /* rna */
-       /* used to define frame range */
-       RNA_def_int(ot->srna, "xmin", 0, INT_MIN, INT_MAX, "X Min", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "xmax", 0, INT_MIN, INT_MAX, "X Max", "", INT_MIN, INT_MAX);
-       /* these are not used, but are needed by borderselect gesture operator stuff */
-       RNA_def_int(ot->srna, "ymin", 0, INT_MIN, INT_MAX, "Y Min", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "ymax", 0, INT_MIN, INT_MAX, "Y Max", "", INT_MIN, INT_MAX);
+       /* used to define frame range.
+        *
+        * note: border Y values are not used,
+        * but are needed by borderselect gesture operator stuff */
+       WM_operator_properties_border(ot);
 }
 
 /* ****************** clear preview range operator ****************************/
index 244a4ea15dee1199dd6b56bca911c6fafa43ad02..c70f661cb14b6b60c8c0283c6e3012ad4ce7482a 100644 (file)
@@ -1178,11 +1178,7 @@ static void VIEW2D_OT_zoom_border(wmOperatorType *ot)
        ot->poll = view_zoom_poll;
        
        /* rna */
-       RNA_def_int(ot->srna, "gesture_mode", 0, INT_MIN, INT_MAX, "Gesture Mode", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "xmin", 0, INT_MIN, INT_MAX, "X Min", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "xmax", 0, INT_MIN, INT_MAX, "X Max", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "ymin", 0, INT_MIN, INT_MAX, "Y Min", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "ymax", 0, INT_MIN, INT_MAX, "Y Max", "", INT_MIN, INT_MAX);
+       WM_operator_properties_gesture_border(ot, FALSE);
 }
 
 /* ********************************************************* */
index b4b2a4ec4f0cf2b6a1fda8891f1abb4a15c8b7c6..285306ce12f2bcfb8d2caa2930c5714de679f9af 100644 (file)
@@ -414,10 +414,7 @@ static int border_select_exec(bContext *C, wmOperator *op)
        int change = FALSE, mode, extend;
 
        /* get rectangle from operator */
-       rect.xmin = RNA_int_get(op->ptr, "xmin");
-       rect.ymin = RNA_int_get(op->ptr, "ymin");
-       rect.xmax = RNA_int_get(op->ptr, "xmax");
-       rect.ymax = RNA_int_get(op->ptr, "ymax");
+       WM_operator_properties_border_to_rcti(op, &rect);
 
        ED_mask_point_pos(sa, ar, rect.xmin, rect.ymin, &rectf.xmin, &rectf.ymin);
        ED_mask_point_pos(sa, ar, rect.xmax, rect.ymax, &rectf.xmax, &rectf.ymax);
index aa49389db724e458f51c9f15ee1c550e5ecd7bbe..e92f7b7a3c217c6b803f6343c50673867453941f 100644 (file)
@@ -3262,11 +3262,7 @@ static void SCREEN_OT_border_select(wmOperatorType *ot)
        
        /* rna */
        RNA_def_int(ot->srna, "event_type", 0, INT_MIN, INT_MAX, "Event Type", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "xmin", 0, INT_MIN, INT_MAX, "X Min", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "xmax", 0, INT_MIN, INT_MAX, "X Max", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "ymin", 0, INT_MIN, INT_MAX, "Y Min", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "ymax", 0, INT_MIN, INT_MAX, "Y Max", "", INT_MIN, INT_MAX);
-       
+       WM_operator_properties_border(ot);
 }
 #endif
 
index 89c328e71d8366142a61dafc6b68558b4f1cd197..e809267d076162c8e87af81620b5048bb0f4abd3 100644 (file)
@@ -398,8 +398,5 @@ void PAINT_OT_hide_show(struct wmOperatorType *ot)
        RNA_def_enum(ot->srna, "area", area_items, PARTIALVIS_INSIDE,
                     "Area", "Which vertices to hide or show");
        
-       RNA_def_int(ot->srna, "xmin", 0, INT_MIN, INT_MAX, "X Min", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "xmax", 0, INT_MIN, INT_MAX, "X Max", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "ymin", 0, INT_MIN, INT_MAX, "Y Min", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "ymax", 0, INT_MIN, INT_MAX, "Y Max", "", INT_MIN, INT_MAX);
+       WM_operator_properties_border(ot);
 }
index 4da283733b07ed73606cab395e639edbc177b9e7..6b61990d4854234e8346c464fae0ac2a089dcea0 100644 (file)
@@ -294,10 +294,7 @@ static int actkeys_borderselect_exec(bContext *C, wmOperator *op)
                deselect_action_keys(&ac, 1, SELECT_SUBTRACT);
        
        /* get settings from operator */
-       rect.xmin = RNA_int_get(op->ptr, "xmin");
-       rect.ymin = RNA_int_get(op->ptr, "ymin");
-       rect.xmax = RNA_int_get(op->ptr, "xmax");
-       rect.ymax = RNA_int_get(op->ptr, "ymax");
+       WM_operator_properties_border_to_rcti(op, &rect);
                
        gesture_mode = RNA_int_get(op->ptr, "gesture_mode");
        if (gesture_mode == GESTURE_MODAL_SELECT)
index c0b11dda5c7aefe64264327ad56331f21b08ca7c..a2cb812d51cc6cf80dfb3f87ed742948f5884c70 100644 (file)
@@ -353,10 +353,7 @@ static int border_select_graph_exec(bContext *C, wmOperator *op)
        rcti rect;
 
        /* get rectangle from operator */
-       rect.xmin = RNA_int_get(op->ptr, "xmin");
-       rect.ymin = RNA_int_get(op->ptr, "ymin");
-       rect.xmax = RNA_int_get(op->ptr, "xmax");
-       rect.ymax = RNA_int_get(op->ptr, "ymax");
+       WM_operator_properties_border_to_rcti(op, &rect);
 
        UI_view2d_region_to_view(&ar->v2d, rect.xmin, rect.ymin, &userdata.rect.xmin, &userdata.rect.ymin);
        UI_view2d_region_to_view(&ar->v2d, rect.xmax, rect.ymax, &userdata.rect.xmax, &userdata.rect.ymax);
index 559fe8c840dd86dc7e5ce0e70fa38085be87a06b..ea3d12269378b41f3593ce850f7d2fa8a40a35a6 100644 (file)
@@ -348,10 +348,7 @@ static int border_select_exec(bContext *C, wmOperator *op)
        int framenr = ED_space_clip_get_clip_frame_number(sc);
 
        /* get rectangle from operator */
-       rect.xmin = RNA_int_get(op->ptr, "xmin");
-       rect.ymin = RNA_int_get(op->ptr, "ymin");
-       rect.xmax = RNA_int_get(op->ptr, "xmax");
-       rect.ymax = RNA_int_get(op->ptr, "ymax");
+       WM_operator_properties_border_to_rcti(op, &rect);
 
        ED_clip_point_stable_pos(sc, ar, rect.xmin, rect.ymin, &rectf.xmin, &rectf.ymin);
        ED_clip_point_stable_pos(sc, ar, rect.xmax, rect.ymax, &rectf.xmax, &rectf.ymax);
index 7630143acd1ad9882471bdc03ec9a519c4a2c4a2..019628535560d05f580bb1f195bd0dfabb2168a1 100644 (file)
@@ -243,10 +243,7 @@ static int file_border_select_modal(bContext *C, wmOperator *op, wmEvent *event)
 
        if (result == OPERATOR_RUNNING_MODAL) {
 
-               rect.xmin = RNA_int_get(op->ptr, "xmin");
-               rect.ymin = RNA_int_get(op->ptr, "ymin");
-               rect.xmax = RNA_int_get(op->ptr, "xmax");
-               rect.ymax = RNA_int_get(op->ptr, "ymax");
+               WM_operator_properties_border_to_rcti(op, &rect);
 
                BLI_rcti_isect(&(ar->v2d.mask), &rect, &rect);
 
@@ -277,10 +274,7 @@ static int file_border_select_exec(bContext *C, wmOperator *op)
        int extend = RNA_boolean_get(op->ptr, "extend");
        short select = (RNA_int_get(op->ptr, "gesture_mode") == GESTURE_MODAL_SELECT);
 
-       rect.xmin = RNA_int_get(op->ptr, "xmin");
-       rect.ymin = RNA_int_get(op->ptr, "ymin");
-       rect.xmax = RNA_int_get(op->ptr, "xmax");
-       rect.ymax = RNA_int_get(op->ptr, "ymax");
+       WM_operator_properties_border_to_rcti(op, &rect);
 
        if (!extend) {
                SpaceFile *sfile = CTX_wm_space_file(C);
index 109241ac3666d9d8035cc40bfe915f008195a822..781099f6a68a6a39e8fecb72e1034729e5e2f27c 100644 (file)
@@ -316,10 +316,7 @@ static int graphkeys_borderselect_exec(bContext *C, wmOperator *op)
        incl_handles = RNA_boolean_get(op->ptr, "include_handles");
        
        /* get settings from operator */
-       rect.xmin = RNA_int_get(op->ptr, "xmin");
-       rect.ymin = RNA_int_get(op->ptr, "ymin");
-       rect.xmax = RNA_int_get(op->ptr, "xmax");
-       rect.ymax = RNA_int_get(op->ptr, "ymax");
+       WM_operator_properties_border_to_rcti(op, &rect);
        
        /* selection 'mode' depends on whether borderselect region only matters on one axis */
        if (RNA_boolean_get(op->ptr, "axis_range")) {
index 46cd253006379414ee083a75d6fef28499cc5c23..2324f735a33654de19aa8c52783d4dcac4f0f3af 100644 (file)
@@ -231,10 +231,7 @@ static int borderselect_exec(bContext *C, wmOperator *op)
        short selecting = (RNA_int_get(op->ptr, "gesture_mode") == GESTURE_MODAL_SELECT);
        //int mval[2];
 
-       rect.xmin = RNA_int_get(op->ptr, "xmin");
-       rect.ymin = RNA_int_get(op->ptr, "ymin");
-       rect.xmax = RNA_int_get(op->ptr, "xmax");
-       rect.ymax = RNA_int_get(op->ptr, "ymax");
+       WM_operator_properties_border_to_rcti(op, &rect);
 
 #if 0
        mval[0] = rect.xmin;
index 3736f83bce2e4849bbeb0bb670f8fb5c09ed9f5a..b6bef651af3a70b047ef7de7f1fb581cbe2faf5b 100644 (file)
@@ -295,10 +295,7 @@ static int nlaedit_borderselect_exec(bContext *C, wmOperator *op)
                deselect_nla_strips(&ac, DESELECT_STRIPS_TEST, SELECT_SUBTRACT);
 
        /* get settings from operator */
-       rect.xmin = RNA_int_get(op->ptr, "xmin");
-       rect.ymin = RNA_int_get(op->ptr, "ymin");
-       rect.xmax = RNA_int_get(op->ptr, "xmax");
-       rect.ymax = RNA_int_get(op->ptr, "ymax");
+       WM_operator_properties_border_to_rcti(op, &rect);
                
        if (RNA_int_get(op->ptr, "gesture_mode") == GESTURE_MODAL_SELECT)
                selectmode = SELECT_ADD;
index e38dff6032cf90f6bdd36cd56bff452b299f0352..7b90d66c11c252ef649eea5231b0de4907d81658 100644 (file)
@@ -467,12 +467,9 @@ static int node_borderselect_exec(bContext *C, wmOperator *op)
        int gesture_mode = RNA_int_get(op->ptr, "gesture_mode");
        int extend = RNA_boolean_get(op->ptr, "extend");
        
-       rect.xmin = RNA_int_get(op->ptr, "xmin");
-       rect.ymin = RNA_int_get(op->ptr, "ymin");
+       WM_operator_properties_border_to_rcti(op, &rect);
+
        UI_view2d_region_to_view(&ar->v2d, rect.xmin, rect.ymin, &rectf.xmin, &rectf.ymin);
-       
-       rect.xmax = RNA_int_get(op->ptr, "xmax");
-       rect.ymax = RNA_int_get(op->ptr, "ymax");
        UI_view2d_region_to_view(&ar->v2d, rect.xmax, rect.ymax, &rectf.xmax, &rectf.ymax);
        
        for (node = snode->edittree->nodes.first; node; node = node->next) {
index 096713f9cd3e7576012d0129fbb16decacb58c47..8b7cf406642fba3a339b75edaaa341f02ade5660 100644 (file)
@@ -899,12 +899,9 @@ static int outliner_border_select_exec(bContext *C, wmOperator *op)
        rctf rectf;
        int gesture_mode = RNA_int_get(op->ptr, "gesture_mode");
 
-       rect.xmin = RNA_int_get(op->ptr, "xmin");
-       rect.ymin = RNA_int_get(op->ptr, "ymin");
-       UI_view2d_region_to_view(&ar->v2d, rect.xmin, rect.ymin, &rectf.xmin, &rectf.ymin);
+       WM_operator_properties_border_to_rcti(op, &rect);
 
-       rect.xmax = RNA_int_get(op->ptr, "xmax");
-       rect.ymax = RNA_int_get(op->ptr, "ymax");
+       UI_view2d_region_to_view(&ar->v2d, rect.xmin, rect.ymin, &rectf.xmin, &rectf.ymin);
        UI_view2d_region_to_view(&ar->v2d, rect.xmax, rect.ymax, &rectf.xmax, &rectf.ymax);
 
        for (te = soops->tree.first; te; te = te->next) {
index 8aba1cb1bc72d6a24db06fd2ce3fca37ed3555c8..3d57f2f88edf25b6b5248d21edfd217132f9a908 100644 (file)
@@ -860,10 +860,7 @@ static int sequencer_borderselect_exec(bContext *C, wmOperator *op)
        if (ed == NULL)
                return OPERATOR_CANCELLED;
 
-       rect.xmin = RNA_int_get(op->ptr, "xmin");
-       rect.ymin = RNA_int_get(op->ptr, "ymin");
-       rect.xmax = RNA_int_get(op->ptr, "xmax");
-       rect.ymax = RNA_int_get(op->ptr, "ymax");
+       WM_operator_properties_border_to_rcti(op, &rect);
        
        mval[0] = rect.xmin;
        mval[1] = rect.ymin;
index e8612d675a7472072c22819c0a346ed249674b2a..20afe42763d1fd560032b6a89eb0103e82c89e12 100644 (file)
@@ -2471,10 +2471,7 @@ static int render_border_exec(bContext *C, wmOperator *op)
        rctf vb;
 
        /* get border select values using rna */
-       rect.xmin = RNA_int_get(op->ptr, "xmin");
-       rect.ymin = RNA_int_get(op->ptr, "ymin");
-       rect.xmax = RNA_int_get(op->ptr, "xmax");
-       rect.ymax = RNA_int_get(op->ptr, "ymax");
+       WM_operator_properties_border_to_rcti(op, &rect);
 
        /* calculate range */
        ED_view3d_calc_camera_border(scene, ar, v3d, rv3d, &vb, FALSE);
@@ -2528,10 +2525,7 @@ void VIEW3D_OT_render_border(wmOperatorType *ot)
        ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
 
        /* rna */
-       RNA_def_int(ot->srna, "xmin", 0, INT_MIN, INT_MAX, "X Min", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "xmax", 0, INT_MIN, INT_MAX, "X Max", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "ymin", 0, INT_MIN, INT_MAX, "Y Min", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "ymax", 0, INT_MIN, INT_MAX, "Y Max", "", INT_MIN, INT_MAX);
+       WM_operator_properties_border(ot);
 
 }
 /* ********************* Border Zoom operator ****************** */
@@ -2562,10 +2556,7 @@ static int view3d_zoom_border_exec(bContext *C, wmOperator *op)
        view3d_operator_needs_opengl(C);
 
        /* get border select values using rna */
-       rect.xmin = RNA_int_get(op->ptr, "xmin");
-       rect.ymin = RNA_int_get(op->ptr, "ymin");
-       rect.xmax = RNA_int_get(op->ptr, "xmax");
-       rect.ymax = RNA_int_get(op->ptr, "ymax");
+       WM_operator_properties_border_to_rcti(op, &rect);
 
        /* check if zooming in/out view */
        gesture_mode = RNA_int_get(op->ptr, "gesture_mode");
@@ -2710,11 +2701,7 @@ void VIEW3D_OT_zoom_border(wmOperatorType *ot)
        ot->flag = 0;
 
        /* rna */
-       RNA_def_int(ot->srna, "gesture_mode", 0, INT_MIN, INT_MAX, "Gesture Mode", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "xmin", 0, INT_MIN, INT_MAX, "X Min", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "xmax", 0, INT_MIN, INT_MAX, "X Max", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "ymin", 0, INT_MIN, INT_MAX, "Y Min", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "ymax", 0, INT_MIN, INT_MAX, "Y Max", "", INT_MIN, INT_MAX);
+       WM_operator_properties_gesture_border(ot, FALSE);
 }
 
 /* sets the view to 1:1 camera/render-pixel */
@@ -3325,10 +3312,7 @@ static int view3d_clipping_exec(bContext *C, wmOperator *op)
        bglMats mats;
        rcti rect;
 
-       rect.xmin = RNA_int_get(op->ptr, "xmin");
-       rect.ymin = RNA_int_get(op->ptr, "ymin");
-       rect.xmax = RNA_int_get(op->ptr, "xmax");
-       rect.ymax = RNA_int_get(op->ptr, "ymax");
+       WM_operator_properties_border_to_rcti(op, &rect);
 
        rv3d->rflag |= RV3D_CLIPPING;
        rv3d->clipbb = MEM_callocN(sizeof(BoundBox), "clipbb");
@@ -3381,10 +3365,7 @@ void VIEW3D_OT_clip_border(wmOperatorType *ot)
        ot->flag = 0;
 
        /* rna */
-       RNA_def_int(ot->srna, "xmin", 0, INT_MIN, INT_MAX, "X Min", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "xmax", 0, INT_MIN, INT_MAX, "X Max", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "ymin", 0, INT_MIN, INT_MAX, "Y Min", "", INT_MIN, INT_MAX);
-       RNA_def_int(ot->srna, "ymax", 0, INT_MIN, INT_MAX, "Y Max", "", INT_MIN, INT_MAX);
+       WM_operator_properties_border(ot);
 }
 
 /* ***************** 3d cursor cursor op ******************* */
index 39ec7514a88ce6808989b9e8e6a4565554f67d21..1a91b0aa085e788e8233a039c7a10334e0ca7d79 100644 (file)
@@ -1938,10 +1938,7 @@ static int view3d_borderselect_exec(bContext *C, wmOperator *op)
        view3d_set_viewcontext(C, &vc);
        
        select = (RNA_int_get(op->ptr, "gesture_mode") == GESTURE_MODAL_SELECT);
-       rect.xmin = RNA_int_get(op->ptr, "xmin");
-       rect.ymin = RNA_int_get(op->ptr, "ymin");
-       rect.xmax = RNA_int_get(op->ptr, "xmax");
-       rect.ymax = RNA_int_get(op->ptr, "ymax");
+       WM_operator_properties_border_to_rcti(op, &rect);
        extend = RNA_boolean_get(op->ptr, "extend");
 
        if (vc.obedit) {
index 0b23b65f2509c81b319236fc935cc024c66a9a0d..f88d4becc7418a76a90e294bf44c72f0237db2a5 100644 (file)
@@ -2937,7 +2937,7 @@ int Resize(TransInfo *t, const int mval[2])
                /* vertices in the radius of the brush end */
                /* outside the clipping area               */
                /* XXX HACK - dg */
-               if(t->flag & (T_PROP_EDIT | T_PROP_CONNECTED)) {
+               if (t->flag & (T_PROP_EDIT | T_PROP_CONNECTED)) {
                        clipUVData(t);
                }
        }
@@ -3802,7 +3802,7 @@ int Translation(TransInfo *t, const int UNUSED(mval[2]))
                /* vertices in the radius of the brush end */
                /* outside the clipping area               */
                /* XXX HACK - dg */
-               if(t->flag & (T_PROP_EDIT | T_PROP_CONNECTED)) {
+               if (t->flag & (T_PROP_EDIT | T_PROP_CONNECTED)) {
                        clipUVData(t);
                }
        }
index de1d51b22040ba3cb2c13d536c2bd2b9742b5c7e..e538e76154cfa8573ca9504dcc1b3dbb45f7b636 100644 (file)
@@ -2453,10 +2453,7 @@ static int border_select_exec(bContext *C, wmOperator *op)
        int change, pinned, select, faces, extend;
 
        /* get rectangle from operator */
-       rect.xmin = RNA_int_get(op->ptr, "xmin");
-       rect.ymin = RNA_int_get(op->ptr, "ymin");
-       rect.xmax = RNA_int_get(op->ptr, "xmax");
-       rect.ymax = RNA_int_get(op->ptr, "ymax");
+       WM_operator_properties_border_to_rcti(op, &rect);
                
        UI_view2d_region_to_view(&ar->v2d, rect.xmin, rect.ymin, &rectf.xmin, &rectf.ymin);
        UI_view2d_region_to_view(&ar->v2d, rect.xmax, rect.ymax, &rectf.xmax, &rectf.ymax);
index 60efc3b35416e763dfa82e5ccc45d14e948b4476..ee7b44fb1e92fa5d96f37c9c2051fa7b0fdd08c4 100644 (file)
@@ -208,6 +208,8 @@ void                WM_operator_properties_create(struct PointerRNA *ptr, const char *opstring
 void           WM_operator_properties_create_ptr(struct PointerRNA *ptr, struct wmOperatorType *ot);
 void           WM_operator_properties_free(struct PointerRNA *ptr);
 void           WM_operator_properties_filesel(struct wmOperatorType *ot, int filter, short type, short action, short flag, short display);
+void        WM_operator_properties_border(struct wmOperatorType *ot);
+void        WM_operator_properties_border_to_rcti(struct wmOperator *op, struct rcti *rect);
 void           WM_operator_properties_gesture_border(struct wmOperatorType *ot, int extend);
 void        WM_operator_properties_mouse_select(struct wmOperatorType *ot);
 void           WM_operator_properties_gesture_straightline(struct wmOperatorType *ot, int cursor);
index 685ef467ad39acba77cc5ee4b719969bff4d878c..52436b6564ceb530e30be3a46da5241625a4a748 100644 (file)
@@ -906,16 +906,31 @@ void WM_operator_properties_select_all(wmOperatorType *ot)
        RNA_def_enum(ot->srna, "action", select_all_actions, SEL_TOGGLE, "Action", "Selection action to execute");
 }
 
-void WM_operator_properties_gesture_border(wmOperatorType *ot, int extend)
+void WM_operator_properties_border(wmOperatorType *ot)
 {
-       RNA_def_int(ot->srna, "gesture_mode", 0, INT_MIN, INT_MAX, "Gesture Mode", "", INT_MIN, INT_MAX);
        RNA_def_int(ot->srna, "xmin", 0, INT_MIN, INT_MAX, "X Min", "", INT_MIN, INT_MAX);
        RNA_def_int(ot->srna, "xmax", 0, INT_MIN, INT_MAX, "X Max", "", INT_MIN, INT_MAX);
        RNA_def_int(ot->srna, "ymin", 0, INT_MIN, INT_MAX, "Y Min", "", INT_MIN, INT_MAX);
        RNA_def_int(ot->srna, "ymax", 0, INT_MIN, INT_MAX, "Y Max", "", INT_MIN, INT_MAX);
+}
+
+void WM_operator_properties_border_to_rcti(struct wmOperator *op, rcti *rect)
+{
+       rect->xmin = RNA_int_get(op->ptr, "xmin");
+       rect->ymin = RNA_int_get(op->ptr, "ymin");
+       rect->xmax = RNA_int_get(op->ptr, "xmax");
+       rect->ymax = RNA_int_get(op->ptr, "ymax");
+}
 
-       if (extend)
+void WM_operator_properties_gesture_border(wmOperatorType *ot, int extend)
+{
+       RNA_def_int(ot->srna, "gesture_mode", 0, INT_MIN, INT_MAX, "Gesture Mode", "", INT_MIN, INT_MAX);
+
+       WM_operator_properties_border(ot);
+
+       if (extend) {
                RNA_def_boolean(ot->srna, "extend", 1, "Extend", "Extend selection instead of deselecting everything first");
+       }
 }
 
 void WM_operator_properties_mouse_select(wmOperatorType *ot)