From f190f2d267df46d25d74009a3c14ac10753d57ca Mon Sep 17 00:00:00 2001 From: Germano Cavalcante Date: Thu, 18 Nov 2021 14:19:59 -0300 Subject: Revert fixes --- source/blender/editors/transform/transform.c | 64 +--- source/blender/editors/transform/transform.h | 12 +- .../editors/transform/transform_convert_node.c | 16 +- .../blender/editors/transform/transform_generics.c | 72 +---- source/blender/editors/transform/transform_input.c | 41 --- source/blender/editors/transform/transform_ops.c | 331 --------------------- source/blender/editors/transform/transform_snap.c | 190 ++---------- source/blender/editors/transform/transform_snap.h | 4 - .../editors/transform/transform_snap_object.c | 36 +-- 9 files changed, 62 insertions(+), 704 deletions(-) (limited to 'source/blender/editors/transform') diff --git a/source/blender/editors/transform/transform.c b/source/blender/editors/transform/transform.c index 51167b327d6..ae4c3f02c46 100644 --- a/source/blender/editors/transform/transform.c +++ b/source/blender/editors/transform/transform.c @@ -108,6 +108,9 @@ void setTransformViewMatrices(TransInfo *t) unit_m4(t->persinv); t->persp = RV3D_ORTHO; } + + calculateCenter2D(t); + calculateCenterLocal(t, t->center_global); } void setTransformViewAspect(TransInfo *t, float r_aspect[3]) @@ -575,21 +578,6 @@ static void viewRedrawPost(bContext *C, TransInfo *t) static bool transform_modal_item_poll(const wmOperator *op, int value) { const TransInfo *t = op->customdata; - if (t->modifiers & MOD_EDIT_SNAP_SOURCE) { - if (value == TFM_MODAL_EDIT_SNAP_SOURCE) { - return true; - } - else if (!ELEM(value, - TFM_MODAL_CANCEL, - TFM_MODAL_CONFIRM, - TFM_MODAL_SNAP_INV_ON, - TFM_MODAL_SNAP_INV_OFF, - TFM_MODAL_ADD_SNAP, - TFM_MODAL_REMOVE_SNAP)) { - return false; - } - } - switch (value) { case TFM_MODAL_CANCEL: { /* TODO: Canceling with LMB is not possible when the operator is activated @@ -605,13 +593,6 @@ static bool transform_modal_item_poll(const wmOperator *op, int value) } break; } - case TFM_MODAL_SNAP_INV_ON: - case TFM_MODAL_SNAP_INV_OFF: { - if (t->modifiers & MOD_SNAP_TEMP) { - return false; - } - break; - } case TFM_MODAL_ADD_SNAP: case TFM_MODAL_REMOVE_SNAP: { if (t->spacetype != SPACE_VIEW3D) { @@ -683,19 +664,6 @@ static bool transform_modal_item_poll(const wmOperator *op, int value) } break; } - case TFM_MODAL_EDIT_SNAP_SOURCE: { - if (t->spacetype != SPACE_VIEW3D) { - return false; - } - if (t->flag & T_RELEASE_CONFIRM) { - return false; - } - if (!ELEM(t->mode, TFM_TRANSLATION, TFM_ROTATION, TFM_RESIZE)) { - /* More modes can be added over time if this feature proves useful for them. */ - return false; - } - break; - } } return true; } @@ -746,7 +714,6 @@ wmKeyMap *transform_modal_keymap(wmKeyConfig *keyconf) {TFM_MODAL_AUTOCONSTRAINT, "AUTOCONSTRAIN", 0, "Automatic Constraint", ""}, {TFM_MODAL_AUTOCONSTRAINTPLANE, "AUTOCONSTRAINPLANE", 0, "Automatic Constraint Plane", ""}, {TFM_MODAL_PRECISION, "PRECISION", 0, "Precision Mode", ""}, - {TFM_MODAL_EDIT_SNAP_SOURCE, "EDIT_SNAP_SOURCE", 0, "Snap Source Toggle", ""}, {0, NULL, 0, NULL, NULL}, }; @@ -899,6 +866,8 @@ int transformEvent(TransInfo *t, const wmEvent *event) t->con.mode |= CON_SELECT; } + copy_v2_v2_int(t->mval, event->mval); + /* Use this for soft redraw. Might cause flicker in object mode */ // t->redraw |= TREDRAW_SOFT; t->redraw |= TREDRAW_HARD; @@ -922,12 +891,7 @@ int transformEvent(TransInfo *t, const wmEvent *event) handled = true; break; case TFM_MODAL_CONFIRM: - if (t->modifiers & MOD_EDIT_SNAP_SOURCE) { - tranform_snap_source_mod_confirm(t); - } - else { - t->state = TRANS_CONFIRM; - } + t->state = TRANS_CONFIRM; handled = true; break; case TFM_MODAL_TRANSLATE: @@ -1192,9 +1156,6 @@ int transformEvent(TransInfo *t, const wmEvent *event) t->redraw |= TREDRAW_HARD; } break; - case TFM_MODAL_EDIT_SNAP_SOURCE: - tranform_snap_source_mod_toggle(t); - break; /* Those two are only handled in transform's own handler, see T44634! */ case TFM_MODAL_EDGESLIDE_UP: case TFM_MODAL_EDGESLIDE_DOWN: @@ -1571,7 +1532,7 @@ void saveTransform(bContext *C, TransInfo *t, wmOperator *op) if (t->flag & T_MODAL) { /* do we check for parameter? */ - if (transformModeUseSnap(t) && !(t->modifiers & MOD_SNAP_TEMP)) { + if (transformModeUseSnap(t)) { if (!(t->modifiers & MOD_SNAP) != !(ts->snap_flag & SCE_SNAP)) { if (t->modifiers & MOD_SNAP) { ts->snap_flag |= SCE_SNAP; @@ -1959,17 +1920,14 @@ void transformApply(bContext *C, TransInfo *t) if ((t->redraw & TREDRAW_HARD) || (t->draw_handle_apply == NULL && (t->redraw & TREDRAW_SOFT))) { selectConstraint(t); - if (t->modifiers & MOD_EDIT_SNAP_SOURCE) { - tranform_snap_source_mod_update(t); - } - else if (t->transform) { + if (t->transform) { t->transform(t, t->mval); /* calls recalcData() */ + viewRedrawForce(C, t); } + t->redraw = TREDRAW_NOTHING; } - - if (t->redraw & (TREDRAW_HARD | TREDRAW_SOFT)) { + else if (t->redraw & TREDRAW_SOFT) { viewRedrawForce(C, t); - t->redraw = TREDRAW_NOTHING; } /* If auto confirm is on, break after one pass */ diff --git a/source/blender/editors/transform/transform.h b/source/blender/editors/transform/transform.h index 9046f1970a3..d78cd13f8b8 100644 --- a/source/blender/editors/transform/transform.h +++ b/source/blender/editors/transform/transform.h @@ -155,9 +155,6 @@ typedef enum { /** No cursor wrapping on region bounds */ T_NO_CURSOR_WRAP = 1 << 23, - - /** To indicate that V3D matrices have changed due to navigation. */ - T_VIEW_DIRTY = 1 << 24, } eTFlag; /** #TransInfo.modifiers */ @@ -166,9 +163,7 @@ typedef enum { MOD_PRECISION = 1 << 1, MOD_SNAP = 1 << 2, MOD_SNAP_INVERT = 1 << 3, - MOD_SNAP_TEMP = 1 << 4, - MOD_CONSTRAINT_SELECT_PLANE = 1 << 5, - MOD_EDIT_SNAP_SOURCE = 1 << 6, + MOD_CONSTRAINT_SELECT_PLANE = 1 << 4, } eTModifier; /** #TransSnap.status */ @@ -299,8 +294,6 @@ enum { TFM_MODAL_AUTOCONSTRAINTPLANE = 29, TFM_MODAL_PRECISION = 30, - - TFM_MODAL_EDIT_SNAP_SOURCE = 31, }; /** \} */ @@ -768,8 +761,6 @@ void applyMouseInput(struct TransInfo *t, struct MouseInput *mi, const int mval[2], float output[3]); -void transform_input_reset(MouseInput *mi, const int mval[2]); -void transform_input_update(TransInfo *t, const float fac); void setCustomPoints(TransInfo *t, MouseInput *mi, const int start[2], const int end[2]); void setCustomPointsFromDirection(TransInfo *t, MouseInput *mi, const float dir[2]); @@ -802,7 +793,6 @@ void calculateCenter2D(TransInfo *t); void calculateCenterLocal(TransInfo *t, const float center_global[3]); void calculateCenter(TransInfo *t); -void tranformViewUpdate(TransInfo *t); /* API functions for getting center points */ void calculateCenterBound(TransInfo *t, float r_center[3]); diff --git a/source/blender/editors/transform/transform_convert_node.c b/source/blender/editors/transform/transform_convert_node.c index 10c0ee48749..da11666d445 100644 --- a/source/blender/editors/transform/transform_convert_node.c +++ b/source/blender/editors/transform/transform_convert_node.c @@ -164,7 +164,6 @@ void createTransNodeData(TransInfo *t) void flushTransNodes(TransInfo *t) { const float dpi_fac = UI_DPI_FAC; - float offset[2] = {0.0f, 0.0f}; View2DEdgePanData *customdata = (View2DEdgePanData *)t->custom.type.data; @@ -178,16 +177,14 @@ void flushTransNodes(TransInfo *t) t->region->winrct.xmin + t->mval[0], t->region->winrct.ymin + t->mval[1], }; - const rctf rect = t->region->v2d.cur; UI_view2d_edge_pan_apply(t->context, customdata, xy); - if (!BLI_rctf_compare(&rect, &t->region->v2d.cur, FLT_EPSILON)) { - /* Additional offset due to change in view2D rect. */ - BLI_rctf_transform_pt_v(&t->region->v2d.cur, &rect, offset, offset); - t->flag |= T_VIEW_DIRTY; - } } } + /* Initial and current view2D rects for additional transform due to view panning and zooming */ + const rctf *rect_src = &customdata->initial_rect; + const rctf *rect_dst = &t->region->v2d.cur; + FOREACH_TRANS_DATA_CONTAINER (t, tc) { applyGridAbsolute(t); @@ -198,7 +195,10 @@ void flushTransNodes(TransInfo *t) bNode *node = td->extra; float loc[2]; - add_v2_v2v2(loc, td2d->loc, offset); + copy_v2_v2(loc, td2d->loc); + + /* additional offset due to change in view2D rect */ + BLI_rctf_transform_pt_v(rect_dst, rect_src, loc, loc); #ifdef USE_NODE_CENTER loc[0] -= 0.5f * BLI_rctf_size_x(&node->totr); diff --git a/source/blender/editors/transform/transform_generics.c b/source/blender/editors/transform/transform_generics.c index 7846fa93f03..4194fb2a0ad 100644 --- a/source/blender/editors/transform/transform_generics.c +++ b/source/blender/editors/transform/transform_generics.c @@ -679,8 +679,6 @@ void initTransInfo(bContext *C, TransInfo *t, wmOperator *op, const wmEvent *eve } setTransformViewMatrices(t); - calculateCenter2D(t); - calculateCenterLocal(t, t->center_global); initNumInput(&t->num); } @@ -1142,33 +1140,6 @@ static void calculateCenter_FromAround(TransInfo *t, int around, float r_center[ } } -static void calculateZfac(TransInfo *t) -{ - /* ED_view3d_calc_zfac() defines a factor for perspective depth correction, - * used in ED_view3d_win_to_delta() */ - - /* zfac is only used convertViewVec only in cases operator was invoked in RGN_TYPE_WINDOW - * and never used in other cases. - * - * We need special case here as well, since ED_view3d_calc_zfac will crash when called - * for a region different from RGN_TYPE_WINDOW. - */ - if (t->spacetype == SPACE_VIEW3D) { - t->zfac = ED_view3d_calc_zfac(t->region->regiondata, t->center_global, NULL); - } - else if (t->spacetype == SPACE_IMAGE) { - SpaceImage *sima = t->area->spacedata.first; - t->zfac = 1.0f / sima->zoom; - } - else { - View2D *v2d = &t->region->v2d; - /* Get zoom fac the same way as in - * `ui_view2d_curRect_validate_resize` - better keep in sync! */ - const float zoomx = (float)(BLI_rcti_size_x(&v2d->mask) + 1) / BLI_rctf_size_x(&v2d->cur); - t->zfac = 1.0f / zoomx; - } -} - void calculateCenter(TransInfo *t) { if ((t->flag & T_OVERRIDE_CENTER) == 0) { @@ -1203,38 +1174,23 @@ void calculateCenter(TransInfo *t) } } - calculateZfac(t); -} - -/* Called every time the view changes due to navigation. - * Adjusts the mouse position relative to the object. */ -void tranformViewUpdate(TransInfo *t) -{ if (t->spacetype == SPACE_VIEW3D) { - setTransformViewMatrices(t); - - for (int i = 0; i < ARRAY_SIZE(t->orient); i++) { - if (t->orient[i].type == V3D_ORIENT_VIEW) { - copy_m3_m4(t->orient[i].matrix, t->viewinv); - normalize_m3(t->orient[i].matrix); - if (t->orient_curr == i) { - copy_m3_m3(t->spacemtx, t->orient[i].matrix); - invert_m3_m3_safe_ortho(t->spacemtx_inv, t->spacemtx); - } - } + /* ED_view3d_calc_zfac() defines a factor for perspective depth correction, + * used in ED_view3d_win_to_delta() */ + + /* zfac is only used convertViewVec only in cases operator was invoked in RGN_TYPE_WINDOW + * and never used in other cases. + * + * We need special case here as well, since ED_view3d_calc_zfac will crash when called + * for a region different from RGN_TYPE_WINDOW. + */ + if (t->region->regiontype == RGN_TYPE_WINDOW) { + t->zfac = ED_view3d_calc_zfac(t->region->regiondata, t->center_global, NULL); + } + else { + t->zfac = 0.0f; } } - - float fac = 1.0f; - if (ELEM(t->spacetype, SPACE_VIEW3D, SPACE_IMAGE, SPACE_NODE)) { - float zfac_prev = t->zfac; - calculateZfac(t); - fac = zfac_prev / t->zfac; - } - - calculateCenter2D(t); - transform_input_update(t, fac); - t->flag &= ~T_VIEW_DIRTY; } void calculatePropRatio(TransInfo *t) diff --git a/source/blender/editors/transform/transform_input.c b/source/blender/editors/transform/transform_input.c index 5d437e3c1d1..0b46d0b9a13 100644 --- a/source/blender/editors/transform/transform_input.c +++ b/source/blender/editors/transform/transform_input.c @@ -418,10 +418,6 @@ void setInputPostFct(MouseInput *mi, void (*post)(struct TransInfo *t, float val void applyMouseInput(TransInfo *t, MouseInput *mi, const int mval[2], float output[3]) { - if (t->modifiers & MOD_EDIT_SNAP_SOURCE) { - return; - } - double mval_db[2]; if (mi->use_virtual_mval) { @@ -459,41 +455,4 @@ void applyMouseInput(TransInfo *t, MouseInput *mi, const int mval[2], float outp } } -void transform_input_update(TransInfo *t, const float fac) -{ - MouseInput *mi = &t->mouse; - int offset[2], center_2d_int[2] = {mi->center[0], mi->center[1]}; - sub_v2_v2v2_int(offset, mi->imval, center_2d_int); - offset[0] *= fac; - offset[1] *= fac; - t->mouse.factor *= fac; - - center_2d_int[0] = t->center2d[0]; - center_2d_int[1] = t->center2d[1]; - add_v2_v2v2_int(mi->imval, center_2d_int, offset); - - float center_old[2]; - copy_v2_v2(center_old, mi->center); - copy_v2_v2(mi->center, t->center2d); - - if (ELEM(mi->apply, InputAngle, InputAngleSpring)) { - float offset_center[2]; - sub_v2_v2v2(offset_center, mi->center, center_old); - struct InputAngle_Data *data = mi->data; - data->mval_prev[0] += offset_center[0]; - data->mval_prev[1] += offset_center[1]; - } -} - -void transform_input_reset(MouseInput *mi, const int mval[2]) -{ - copy_v2_v2_int(mi->imval, mval); - if (ELEM(mi->apply, InputAngle, InputAngleSpring)) { - struct InputAngle_Data *data = mi->data; - data->mval_prev[0] = mi->imval[0]; - data->mval_prev[1] = mi->imval[1]; - data->angle = 0.0f; - } -} - /** \} */ diff --git a/source/blender/editors/transform/transform_ops.c b/source/blender/editors/transform/transform_ops.c index 37eb85bfda2..5ed340abf97 100644 --- a/source/blender/editors/transform/transform_ops.c +++ b/source/blender/editors/transform/transform_ops.c @@ -23,10 +23,7 @@ #include "DNA_object_types.h" #include "DNA_scene_types.h" -#include "BLI_alloca.h" -#include "BLI_listbase.h" #include "BLI_math.h" -#include "BLI_string.h" #include "BLI_utildefines.h" #include "BLT_translation.h" @@ -424,17 +421,7 @@ static int transform_modal(bContext *C, wmOperator *op, const wmEvent *event) return OPERATOR_PASS_THROUGH; } #endif - if (event->type == MOUSEMOVE) { - copy_v2_v2_int(t->mval, event->mval); - } - if (WM_operator_do_navigation(C, op, event)) { - t->flag |= T_VIEW_DIRTY; - return OPERATOR_RUNNING_MODAL; - } - else if (t->flag & T_VIEW_DIRTY) { - tranformViewUpdate(t); - } /* XXX insert keys are called here, and require context */ t->context = C; exit_code = transformEvent(t, event); @@ -1365,321 +1352,6 @@ static void TRANSFORM_OT_from_gizmo(struct wmOperatorType *ot) ot->invoke = transform_from_gizmo_invoke; } -static wmKeyMapItem **navigation_keymaps(bContext *C, int *r_kmi_len) -{ - wmKeyMapItem *km_items[70]; - int kmi_len = 0; - - const char *op_names[] = { - /* 3D View. */ - "VIEW3D_OT_zoom", - "VIEW3D_OT_rotate", - "VIEW3D_OT_move", - "VIEW3D_OT_view_pan", - "VIEW3D_OT_dolly", - "VIEW3D_OT_view_orbit", - "VIEW3D_OT_view_roll", -#ifdef WITH_INPUT_NDOF - "VIEW3D_OT_ndof_orbit_zoom", - "VIEW3D_OT_ndof_orbit", - "VIEW3D_OT_ndof_pan", - "VIEW3D_OT_ndof_all", -#endif - /* Image. */ - "IMAGE_OT_view_pan", - "IMAGE_OT_view_zoom_in", - "IMAGE_OT_view_zoom_out", -#ifdef WITH_INPUT_NDOF - "IMAGE_OT_view_ndof", -#endif - /* View2D. */ - "VIEW2D_OT_pan", - "VIEW2D_OT_zoom_in", - "VIEW2D_OT_zoom_out", -#ifdef WITH_INPUT_NDOF - "VIEW2D_OT_ndof", -#endif - }; - - wmWindowManager *wm = CTX_wm_manager(C); - wmKeyMap *keymap[3]; - keymap[0] = WM_keymap_find_all(wm, "3D View", SPACE_VIEW3D, 0); - keymap[1] = WM_keymap_find_all(wm, "Image", SPACE_IMAGE, 0); - keymap[2] = WM_keymap_find_all(wm, "View2D", 0, 0); - - for (int i = 0; i < 3; i++) { - LISTBASE_FOREACH (wmKeyMapItem *, kmi, &keymap[i]->items) { - if (!(STRPREFIX(kmi->idname, "VIEW") || STRPREFIX(kmi->idname, "IMAGE"))) { - continue; - } - if (kmi->flag & KMI_INACTIVE) { - continue; - } - for (int i = 0; i < ARRAY_SIZE(op_names); i++) { - if (STREQ(kmi->idname, op_names[i])) { - km_items[kmi_len++] = kmi; - break; - } - } - if (kmi_len == ARRAY_SIZE(km_items)) { - BLI_assert(false); - break; - } - } - } - size_t km_items_size = sizeof(void *) * kmi_len; - wmKeyMapItem **r_km_items = MEM_mallocN(km_items_size, __func__); - memcpy(r_km_items, km_items, km_items_size); - - *r_kmi_len = kmi_len; - return r_km_items; -} - -static bool kmi_cmp(const wmKeyMapItem *kmi_a, const wmKeyMapItem *kmi_b) -{ - if (kmi_a->shift != kmi_b->shift) { - return false; - } - if (kmi_a->ctrl != kmi_b->ctrl) { - return false; - } - if (kmi_a->alt != kmi_b->alt) { - return false; - } - if (kmi_a->oskey != kmi_b->oskey) { - return false; - } - if (!ELEM(kmi_b->type, kmi_a->type, KM_ANY)) { - bool is_wheelin_modal = ELEM(kmi_b->type, WHEELUPMOUSE, WHEELINMOUSE); - bool is_wheelout_modal = ELEM(kmi_b->type, WHEELDOWNMOUSE, WHEELOUTMOUSE); - bool is_wheelin = ELEM(kmi_a->type, WHEELUPMOUSE, WHEELINMOUSE); - bool is_wheelout = ELEM(kmi_a->type, WHEELDOWNMOUSE, WHEELOUTMOUSE); - if (!(is_wheelin_modal || is_wheelout_modal) || - ((is_wheelin_modal != is_wheelin) || (is_wheelout_modal != is_wheelout))) { - return false; - } - } - if (!ELEM(kmi_b->val, kmi_a->val, KM_ANY)) { - return false; - } - if (kmi_a->keymodifier != kmi_b->keymodifier) { - return false; - } - return true; -} - -static const wmKeyMapItem *modalmap_is_conflicting(wmKeyMapItem **km_items, - const int km_items_len, - wmKeyMapItem *kmi_modal) -{ - for (int i = 0; i < km_items_len; i++) { - const wmKeyMapItem *kmi = km_items[i]; - if (kmi_cmp(kmi, kmi_modal)) { - return kmi; - } - } - return NULL; -} - -#define LABEL_LINE_SIZE 140 -struct KMConflict { - wmKeyMapItem *kmi; - wmKeyMapItem kmi_new; - char descr[LABEL_LINE_SIZE]; -}; - -struct KMConflictOPData { - int conflicts_len; - struct KMConflict conflicts[0]; -}; - -static void modalkeymap_conflict_free(wmOperator *op) -{ - struct KMConflictOPData *data = op->customdata; - MEM_freeN(data); -} - -static int modalkeymap_update_invoke(bContext *C, wmOperator *op, const wmEvent *UNUSED(event)) -{ - int kmi_navigate_len; - wmKeyMapItem **kmi_navigate = navigation_keymaps(C, &kmi_navigate_len); - - struct KMConflict conflicts[10]; - int conflicts_len = 0; - - wmWindowManager *wm = CTX_wm_manager(C); - wmKeyConfig *keyconf = wm->userconf; - wmKeyMap *modalmap = WM_keymap_active(wm, WM_modalkeymap_find(keyconf, "Transform Modal Map")); - LISTBASE_FOREACH (wmKeyMapItem *, kmi_modal, &modalmap->items) { - if (kmi_modal->flag & KMI_INACTIVE) { - continue; - } - const wmKeyMapItem *kmi = modalmap_is_conflicting(kmi_navigate, kmi_navigate_len, kmi_modal); - if (kmi) { - struct KMConflict *conflict = &conflicts[conflicts_len++]; - conflict->kmi = kmi_modal; - conflict->kmi_new = *kmi_modal; - conflict->kmi_new.alt = true; - wmKeyMapItem *kmi_new = &conflict->kmi_new; - - const char *name; - char descriptor[LABEL_LINE_SIZE], descriptor_new[LABEL_LINE_SIZE]; - WM_keymap_item_to_string(kmi_modal, false, descriptor, LABEL_LINE_SIZE); - WM_keymap_item_to_string(kmi_new, false, descriptor_new, LABEL_LINE_SIZE); - RNA_enum_name(modalmap->modal_items, kmi_modal->propvalue, &name); - - BLI_snprintf(conflict->descr, - LABEL_LINE_SIZE, - "\"%s\" will change from '%s' to '%s' (as it conflicts with \"%s\")", - TIP_(name), - descriptor, - descriptor_new, - kmi->idname); - } - } - MEM_freeN(kmi_navigate); - - if (!conflicts_len) { - return OPERATOR_CANCELLED; - } - - struct KMConflictOPData *data; - size_t conflicts_size = sizeof(struct KMConflict) * conflicts_len; - data = MEM_mallocN(sizeof(struct KMConflictOPData) + conflicts_size, __func__); - data->conflicts_len = conflicts_len; - memcpy(data->conflicts, conflicts, conflicts_size); - - op->customdata = data; - return WM_operator_props_dialog_popup(C, op, 750); -} - -static int modalkeymap_exec(bContext *UNUSED(C), wmOperator *op) -{ - struct KMConflictOPData *data = op->customdata; - struct KMConflict *conflicts = data->conflicts; - for (int i = 0; i < data->conflicts_len; i++) { - struct KMConflict *conflict = &conflicts[i]; - *conflict->kmi = conflict->kmi_new; - WM_keyconfig_update_tag(NULL, conflict->kmi); - } - - modalkeymap_conflict_free(op); - if (U.runtime.is_dirty == false) { - U.runtime.is_dirty = true; - WM_main_add_notifier(NC_WINDOW, NULL); - } - return OPERATOR_FINISHED; -} - -static void modalkeymap_cancel(bContext *UNUSED(C), wmOperator *op) -{ - modalkeymap_conflict_free(op); -} - -static void modalkeymap_draw(bContext *UNUSED(C), wmOperator *op) -{ - struct KMConflictOPData *data = op->customdata; - struct KMConflict *conflicts = data->conflicts; - for (int i = 0; i < data->conflicts_len; i++) { - struct KMConflict *conflict = &conflicts[i]; - uiItemL(op->layout, conflict->descr, ICON_DOT); - } -} - -static void TRANSFORM_OT_modalkeymap_update(struct wmOperatorType *ot) -{ - /* identifiers */ - ot->name = "Update Transform Modal Map"; - ot->description = "Edit the Transform Modal Map to avoid conflicts with navigation shortcuts"; - ot->idname = "TRANSFORM_OT_modalkeymap_update"; - ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO; - - /* api callbacks */ - ot->invoke = modalkeymap_update_invoke; - ot->exec = modalkeymap_exec; - ot->cancel = modalkeymap_cancel; - ot->ui = modalkeymap_draw; -} - -static int modalkeymap_restore_invoke(bContext *C, wmOperator *op, const wmEvent *UNUSED(event)) -{ - struct KMConflict conflicts[10]; - int conflicts_len = 0; - - wmWindowManager *wm = CTX_wm_manager(C); - wmKeyMap *modalmap = WM_modalkeymap_find(wm->defaultconf, "Transform Modal Map"); - wmKeyMap *modalmap_act = WM_keymap_active( - wm, WM_modalkeymap_find(wm->userconf, "Transform Modal Map")); - - wmKeyMapItem *kmi_orig = modalmap->items.first, *kmi = modalmap_act->items.first; - while (kmi_orig && kmi) { - if (!kmi_cmp(kmi_orig, kmi)) { - struct KMConflict *conflict = &conflicts[conflicts_len++]; - conflict->kmi = kmi; - conflict->kmi_new = *kmi_orig; - wmKeyMapItem *kmi_new = &conflict->kmi_new; - - const char *name; - char descriptor[LABEL_LINE_SIZE], descriptor_new[LABEL_LINE_SIZE]; - WM_keymap_item_to_string(kmi, false, descriptor, LABEL_LINE_SIZE); - WM_keymap_item_to_string(kmi_new, false, descriptor_new, LABEL_LINE_SIZE); - RNA_enum_name(modalmap->modal_items, kmi->propvalue, &name); - - BLI_snprintf(conflict->descr, - LABEL_LINE_SIZE, - "\"%s\" will be restored from '%s' to '%s'", - TIP_(name), - descriptor, - descriptor_new); - } - kmi_orig = kmi_orig->next; - kmi = kmi->next; - } - - if (!conflicts_len) { - return OPERATOR_CANCELLED; - } - - struct KMConflictOPData *data; - size_t conflicts_size = sizeof(struct KMConflict) * conflicts_len; - data = MEM_mallocN(sizeof(struct KMConflictOPData) + conflicts_size, __func__); - data->conflicts_len = conflicts_len; - memcpy(data->conflicts, conflicts, conflicts_size); - - op->customdata = data; - return WM_operator_props_dialog_popup(C, op, 600); -} - -static int modalkeymap_restore_exec(bContext *C, wmOperator *op) -{ - wmWindowManager *wm = CTX_wm_manager(C); - wmKeyConfig *keyconf = wm->userconf; - wmKeyMap *modalmap = WM_modalkeymap_find(keyconf, "Transform Modal Map"); - WM_keymap_restore_to_default(modalmap, wm); - - modalkeymap_conflict_free(op); - if (U.runtime.is_dirty == false) { - U.runtime.is_dirty = true; - WM_main_add_notifier(NC_WINDOW, NULL); - } - return OPERATOR_FINISHED; -} - -static void TRANSFORM_OT_modalkeymap_restore(struct wmOperatorType *ot) -{ - /* identifiers */ - ot->name = "Restore Transform Modal Map"; - ot->description = "Restore Transform Modal Map to Default"; - ot->idname = "TRANSFORM_OT_modalkeymap_restore"; - ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO; - - /* api callbacks */ - ot->invoke = modalkeymap_restore_invoke; - ot->exec = modalkeymap_restore_exec; - ot->cancel = modalkeymap_cancel; - ot->ui = modalkeymap_draw; -} - void transform_operatortypes(void) { TransformModeItem *tmode; @@ -1695,9 +1367,6 @@ void transform_operatortypes(void) WM_operatortype_append(TRANSFORM_OT_delete_orientation); WM_operatortype_append(TRANSFORM_OT_from_gizmo); - - WM_operatortype_append(TRANSFORM_OT_modalkeymap_update); - WM_operatortype_append(TRANSFORM_OT_modalkeymap_restore); } void ED_keymap_transform(wmKeyConfig *keyconf) diff --git a/source/blender/editors/transform/transform_snap.c b/source/blender/editors/transform/transform_snap.c index 8ed0a1f2ea8..71f26ef0594 100644 --- a/source/blender/editors/transform/transform_snap.c +++ b/source/blender/editors/transform/transform_snap.c @@ -136,10 +136,6 @@ bool validSnap(const TransInfo *t) bool activeSnap(const TransInfo *t) { - if (t->modifiers & MOD_SNAP_TEMP) { - return true; - } - return ((t->modifiers & (MOD_SNAP | MOD_SNAP_INVERT)) == MOD_SNAP) || ((t->modifiers & (MOD_SNAP | MOD_SNAP_INVERT)) == MOD_SNAP_INVERT); } @@ -182,7 +178,7 @@ bool transformModeUseSnap(const TransInfo *t) static bool doForceIncrementSnap(const TransInfo *t) { - return !(t->modifiers & MOD_SNAP_TEMP) && !transformModeUseSnap(t); + return !transformModeUseSnap(t); } void drawSnapping(const struct bContext *C, TransInfo *t) @@ -236,47 +232,6 @@ void drawSnapping(const struct bContext *C, TransInfo *t) imm_drawcircball(p->co, ED_view3d_pixel_size(rv3d, p->co) * size, view_inv, pos); } - if (t->modifiers & MOD_EDIT_SNAP_SOURCE) { - /* Indicate the new snap source position. */ - - float snap_point[3]; - getSnapPoint(t, snap_point); - - float vx[3], vy[3], v[3]; - float size_tmp = ED_view3d_pixel_size(rv3d, snap_point) * size; - float size_fac = 0.5f; - - mul_v3_v3fl(vx, view_inv[0], size_tmp); - mul_v3_v3fl(vy, view_inv[1], size_tmp); - - immUniformColor4ubv(col); - - imm_drawcircball(snap_point, size_tmp, view_inv, pos); - - immBegin(GPU_PRIM_LINES, 8); - add_v3_v3v3(v, snap_point, vx); - immVertex3fv(pos, v); - madd_v3_v3fl(v, vx, size_fac); - immVertex3fv(pos, v); - - sub_v3_v3v3(v, snap_point, vx); - immVertex3fv(pos, v); - madd_v3_v3fl(v, vx, -size_fac); - immVertex3fv(pos, v); - - add_v3_v3v3(v, snap_point, vy); - immVertex3fv(pos, v); - madd_v3_v3fl(v, vy, size_fac); - immVertex3fv(pos, v); - - sub_v3_v3v3(v, snap_point, vy); - immVertex3fv(pos, v); - madd_v3_v3fl(v, vy, -size_fac); - immVertex3fv(pos, v); - - immEnd(); - } - immUnbindProgram(); } @@ -289,7 +244,7 @@ void drawSnapping(const struct bContext *C, TransInfo *t) loc_prev = t->tsnap.snapTarget; } - if (t->tsnap.status & POINT_INIT) { + if (validSnap(t)) { loc_cur = t->tsnap.snapPoint; } @@ -522,7 +477,8 @@ void applyGridAbsolute(TransInfo *t) void applySnapping(TransInfo *t, float *vec) { - if (!transformModeUseSnap(t) && !(t->modifiers & MOD_SNAP_TEMP)) { + /* Each Trans Data already makes the snap to face */ + if (doForceIncrementSnap(t)) { return; } @@ -703,20 +659,6 @@ static short snap_select_type_get(TransInfo *t) return r_snap_select; } -static void snap_object_context_setup(TransInfo *t) -{ - if (t->data_type != TC_MESH_VERTS) { - return; - } - /* Ignore elements being transformed. */ - ED_transform_snap_object_context_set_editmesh_callbacks( - t->tsnap.object_context, - (bool (*)(BMVert *, void *))BM_elem_cb_check_hflag_disabled, - bm_edge_is_snap_target, - bm_face_is_snap_target, - POINTER_FROM_UINT((BM_ELEM_SELECT | BM_ELEM_HIDDEN))); -} - static void initSnappingMode(TransInfo *t) { ToolSettings *ts = t->settings; @@ -728,13 +670,27 @@ static void initSnappingMode(TransInfo *t) t->tsnap.project = 0; } - setSnappingCallback(t); + if (ELEM(t->spacetype, SPACE_VIEW3D, SPACE_IMAGE, SPACE_NODE, SPACE_SEQ)) { + /* Not with camera selected in camera view. */ + if (!(t->options & CTX_CAMERA)) { + setSnappingCallback(t); + } + } if (t->spacetype == SPACE_VIEW3D) { if (t->tsnap.object_context == NULL) { t->tsnap.use_backface_culling = snap_use_backface_culling(t); t->tsnap.object_context = ED_transform_snap_object_context_create(t->scene, 0); - snap_object_context_setup(t); + + if (t->data_type == TC_MESH_VERTS) { + /* Ignore elements being transformed. */ + ED_transform_snap_object_context_set_editmesh_callbacks( + t->tsnap.object_context, + (bool (*)(BMVert *, void *))BM_elem_cb_check_hflag_disabled, + bm_edge_is_snap_target, + bm_face_is_snap_target, + POINTER_FROM_UINT((BM_ELEM_SELECT | BM_ELEM_HIDDEN))); + } } } else if (t->spacetype == SPACE_SEQ) { @@ -819,16 +775,10 @@ void freeSnapping(TransInfo *t) static void setSnappingCallback(TransInfo *t) { if (t->spacetype == SPACE_VIEW3D) { - if (t->options & CTX_CAMERA) { - /* Not with camera selected in camera view. */ - return; - } t->tsnap.calcSnap = snap_calc_view3d_fn; } - else if (t->spacetype == SPACE_IMAGE) { - if (t->obedit_type == OB_MESH) { - t->tsnap.calcSnap = snap_calc_uv_fn; - } + else if (t->spacetype == SPACE_IMAGE && t->obedit_type == OB_MESH) { + t->tsnap.calcSnap = snap_calc_uv_fn; } else if (t->spacetype == SPACE_NODE) { t->tsnap.calcSnap = snap_calc_node_fn; @@ -838,9 +788,6 @@ static void setSnappingCallback(TransInfo *t) /* The target is calculated along with the snap point. */ return; } - else { - return; - } switch (t->tsnap.target) { case SCE_SNAP_TARGET_CLOSEST: @@ -958,15 +905,6 @@ void getSnapPoint(const TransInfo *t, float vec[3]) } } -static void transform_snap_multipoints_free(TransInfo *t) -{ - if (t->tsnap.status & MULTI_POINTS) { - BLI_freelistN(&t->tsnap.points); - t->tsnap.status &= ~MULTI_POINTS; - t->tsnap.selectedPoint = NULL; - } -} - /** \} */ /* -------------------------------------------------------------------- */ @@ -1681,90 +1619,6 @@ bool transform_snap_increment(const TransInfo *t, float *r_val) return transform_snap_increment_ex(t, false, r_val); } -static void snap_source_mod_start(TransInfo *t) -{ - t->modifiers |= MOD_EDIT_SNAP_SOURCE; - t->tsnap.modeSelect = SNAP_ALL; - if ((t->tsnap.status & TARGET_INIT) == 0) { - /* Calculate the current target for the perpendicular snap. */ - t->tsnap.targetSnap(t); - - /* Fallback. */ - TargetSnapMedian(t); - } - t->tsnap.targetSnap = NULL; - - if ((t->tsnap.mode & ~(SCE_SNAP_MODE_INCREMENT | SCE_SNAP_MODE_GRID)) == 0) { - /* Initialize snap modes for geometry. */ - t->tsnap.mode &= ~(SCE_SNAP_MODE_INCREMENT | SCE_SNAP_MODE_GRID); - t->tsnap.mode |= SCE_SNAP_MODE_GEOM; - } - - if (!activeSnap(t) || !transformModeUseSnap(t)) { - t->modifiers |= MOD_SNAP_TEMP; - } - - restoreTransObjects(t); - - if (t->data_type == TC_MESH_VERTS) { - ED_transform_snap_object_context_set_editmesh_callbacks( - t->tsnap.object_context, NULL, NULL, NULL, NULL); - } - - t->redraw |= TREDRAW_SOFT; -} - -static void snap_source_mod_end(TransInfo *t) -{ - t->modifiers &= ~MOD_EDIT_SNAP_SOURCE; - - /* Restore. */ - t->tsnap.modeSelect = snap_select_type_get(t); - snap_object_context_setup(t); - transform_snap_multipoints_free(t); -} - -void tranform_snap_source_mod_toggle(TransInfo *t) -{ - if (t->flag & T_RELEASE_CONFIRM) { - return; - } - - if (!(t->modifiers & MOD_EDIT_SNAP_SOURCE)) { - snap_source_mod_start(t); - } - else if (t->modifiers & MOD_EDIT_SNAP_SOURCE) { - /* Cancel. */ - t->modifiers &= ~MOD_SNAP_TEMP; - t->tsnap.mode = snap_mode_from_scene(t); - setSnappingCallback(t); - snap_source_mod_end(t); - } -} - -void tranform_snap_source_mod_update(TransInfo *t) -{ - BLI_assert(t->modifiers & MOD_EDIT_SNAP_SOURCE); - - /* Time base quirky code to go around findnearest slowness */ - /* TODO: add exception for object mode, no need to slow it down then. */ - double current = PIL_check_seconds_timer(); - if (current - t->tsnap.last >= 0.01) { - t->tsnap.calcSnap(t, NULL); - t->tsnap.last = current; - t->redraw |= TREDRAW_SOFT; - } -} - -void tranform_snap_source_mod_confirm(TransInfo *t) -{ - BLI_assert(t->modifiers & MOD_EDIT_SNAP_SOURCE); - getSnapPoint(t, t->tsnap.snapTarget); - - transform_input_reset(&t->mouse, t->mval); - snap_source_mod_end(t); -} - /** \} */ /* -------------------------------------------------------------------- */ diff --git a/source/blender/editors/transform/transform_snap.h b/source/blender/editors/transform/transform_snap.h index af13dac7ee7..ed7f93304bc 100644 --- a/source/blender/editors/transform/transform_snap.h +++ b/source/blender/editors/transform/transform_snap.h @@ -52,10 +52,6 @@ bool transform_snap_increment_ex(const TransInfo *t, bool use_local_space, float bool transform_snap_increment(const TransInfo *t, float *val); bool transform_snap_grid(TransInfo *t, float *val); -void tranform_snap_source_mod_toggle(TransInfo *t); -void tranform_snap_source_mod_update(TransInfo *t); -void tranform_snap_source_mod_confirm(TransInfo *t); - bool activeSnap(const TransInfo *t); bool activeSnap_with_project(const TransInfo *t); diff --git a/source/blender/editors/transform/transform_snap_object.c b/source/blender/editors/transform/transform_snap_object.c index 9f05a0a9562..4b981e763f1 100644 --- a/source/blender/editors/transform/transform_snap_object.c +++ b/source/blender/editors/transform/transform_snap_object.c @@ -391,8 +391,9 @@ static SnapObjectData *snap_object_data_editmesh_get(SnapObjectContext *sctx, } else if (sod->mesh_runtime) { if (sod->mesh_runtime != snap_object_data_editmesh_runtime_get(ob_eval)) { - if (G.moving && !sod->treedata_editmesh.cached && !sod->cached[0] && !sod->cached[1]) { + if (G.moving) { /* Hack to avoid updating while transforming. */ + BLI_assert(!sod->treedata_editmesh.cached && !sod->cached[0] && !sod->cached[1]); sod->mesh_runtime = snap_object_data_editmesh_runtime_get(ob_eval); } else { @@ -2829,15 +2830,6 @@ void ED_transform_snap_object_context_destroy(SnapObjectContext *sctx) MEM_freeN(sctx); } -static void transform_snap_object_context_cache_clear(SnapObjectContext *sctx) -{ - BLI_ghash_clear(sctx->cache.object_map, NULL, snap_object_data_free); - if (sctx->cache.data_to_object_map != NULL) { - BLI_ghash_clear(sctx->cache.data_to_object_map, NULL, NULL); - } - BLI_memarena_clear(sctx->cache.mem_arena); -} - void ED_transform_snap_object_context_set_editmesh_callbacks( SnapObjectContext *sctx, bool (*test_vert_fn)(BMVert *, void *user_data), @@ -2845,27 +2837,11 @@ void ED_transform_snap_object_context_set_editmesh_callbacks( bool (*test_face_fn)(BMFace *, void *user_data), void *user_data) { - bool is_cache_dirty = false; - if (sctx->callbacks.edit_mesh.test_vert_fn != test_vert_fn) { - sctx->callbacks.edit_mesh.test_vert_fn = test_vert_fn; - is_cache_dirty = true; - } - if (sctx->callbacks.edit_mesh.test_edge_fn != test_edge_fn) { - sctx->callbacks.edit_mesh.test_edge_fn = test_edge_fn; - is_cache_dirty = true; - } - if (sctx->callbacks.edit_mesh.test_face_fn != test_face_fn) { - sctx->callbacks.edit_mesh.test_face_fn = test_face_fn; - is_cache_dirty = true; - } - if (sctx->callbacks.edit_mesh.user_data != user_data) { - sctx->callbacks.edit_mesh.user_data = user_data; - is_cache_dirty = true; - } + sctx->callbacks.edit_mesh.test_vert_fn = test_vert_fn; + sctx->callbacks.edit_mesh.test_edge_fn = test_edge_fn; + sctx->callbacks.edit_mesh.test_face_fn = test_face_fn; - if (is_cache_dirty) { - transform_snap_object_context_cache_clear(sctx); - } + sctx->callbacks.edit_mesh.user_data = user_data; } bool ED_transform_snap_object_project_ray_ex(SnapObjectContext *sctx, -- cgit v1.2.3