From d421adb83e8d39bd9d74ea3d526326b70e93c341 Mon Sep 17 00:00:00 2001 From: Campbell Barton Date: Tue, 18 Sep 2018 16:10:07 +1000 Subject: Gizmo: de-duplicate poll logic Checking the active tool or operator was a common way to check if the gizmo was still in use. --- source/blender/editors/space_view3d/view3d_gizmo_ruler.c | 15 ++------------- 1 file changed, 2 insertions(+), 13 deletions(-) (limited to 'source/blender/editors/space_view3d/view3d_gizmo_ruler.c') diff --git a/source/blender/editors/space_view3d/view3d_gizmo_ruler.c b/source/blender/editors/space_view3d/view3d_gizmo_ruler.c index 6bb6a022a82..62f66334516 100644 --- a/source/blender/editors/space_view3d/view3d_gizmo_ruler.c +++ b/source/blender/editors/space_view3d/view3d_gizmo_ruler.c @@ -45,6 +45,7 @@ #include "BIF_gl.h" +#include "ED_gizmo_utils.h" #include "ED_gpencil.h" #include "ED_screen.h" #include "ED_transform_snap_object_context.h" @@ -975,18 +976,6 @@ void VIEW3D_GT_ruler_item(wmGizmoType *gzt) /** \name Ruler Gizmo Group * \{ */ -static bool WIDGETGROUP_ruler_poll(const bContext *C, wmGizmoGroupType *gzgt) -{ - bToolRef_Runtime *tref_rt = WM_toolsystem_runtime_from_context((bContext *)C); - if ((tref_rt == NULL) || - !STREQ(gzgt->idname, tref_rt->gizmo_group)) - { - WM_gizmo_group_type_unlink_delayed_ptr(gzgt); - return false; - } - return true; -} - static void WIDGETGROUP_ruler_setup(const bContext *C, wmGizmoGroup *gzgroup) { RulerInfo *ruler_info = MEM_callocN(sizeof(RulerInfo), __func__); @@ -1015,7 +1004,7 @@ void VIEW3D_GGT_ruler(wmGizmoGroupType *gzgt) gzgt->gzmap_params.spaceid = SPACE_VIEW3D; gzgt->gzmap_params.regionid = RGN_TYPE_WINDOW; - gzgt->poll = WIDGETGROUP_ruler_poll; + gzgt->poll = ED_gizmo_poll_or_unlink_delayed_from_tool; gzgt->setup = WIDGETGROUP_ruler_setup; } -- cgit v1.2.3