[Bf-blender-cvs] [b8c7ff564cf] master: Cleanup: un-indent ED_screen_set_active_region

Campbell Barton noreply at git.blender.org
Fri Feb 21 16:44:42 CET 2020


Commit: b8c7ff564cf580e72f1647e95fe6e02352aad3d2
Author: Campbell Barton
Date:   Sat Feb 22 02:41:49 2020 +1100
Branches: master
https://developer.blender.org/rBb8c7ff564cf580e72f1647e95fe6e02352aad3d2

Cleanup: un-indent ED_screen_set_active_region

Also remove redundant flag assignment from previous comment.

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

M	source/blender/editors/screen/screen_edit.c
M	source/blender/windowmanager/intern/wm_toolsystem.c

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

diff --git a/source/blender/editors/screen/screen_edit.c b/source/blender/editors/screen/screen_edit.c
index 427f2f085c6..61affe3896b 100644
--- a/source/blender/editors/screen/screen_edit.c
+++ b/source/blender/editors/screen/screen_edit.c
@@ -679,95 +679,98 @@ static void screen_cursor_set(wmWindow *win, const int xy[2])
   }
 }
 
-/* called in wm_event_system.c. sets state vars in screen, cursors */
-/* event type is mouse move */
+/**
+ * Called in wm_event_system.c. sets state vars in screen, cursors.
+ * event type is mouse move.
+ */
 void ED_screen_set_active_region(bContext *C, wmWindow *win, const int xy[2])
 {
   bScreen *scr = WM_window_get_active_screen(win);
+  if (scr == NULL) {
+    return;
+  }
 
-  if (scr) {
-    ScrArea *sa = NULL;
-    ARegion *ar;
-    ARegion *old_ar = scr->active_region;
+  ScrArea *sa = NULL;
+  ARegion *ar;
+  ARegion *old_ar = scr->active_region;
 
-    ED_screen_areas_iter(win, scr, area_iter)
-    {
-      if (xy[0] > area_iter->totrct.xmin && xy[0] < area_iter->totrct.xmax) {
-        if (xy[1] > area_iter->totrct.ymin && xy[1] < area_iter->totrct.ymax) {
-          if (ED_area_azones_update(area_iter, xy) == NULL) {
-            sa = area_iter;
-            break;
-          }
-        }
-      }
-    }
-    if (sa) {
-      /* make overlap active when mouse over */
-      for (ar = sa->regionbase.first; ar; ar = ar->next) {
-        if (ED_region_contains_xy(ar, xy)) {
-          scr->active_region = ar;
+  ED_screen_areas_iter(win, scr, area_iter)
+  {
+    if (xy[0] > area_iter->totrct.xmin && xy[0] < area_iter->totrct.xmax) {
+      if (xy[1] > area_iter->totrct.ymin && xy[1] < area_iter->totrct.ymax) {
+        if (ED_area_azones_update(area_iter, xy) == NULL) {
+          sa = area_iter;
           break;
         }
       }
     }
-    else {
-      scr->active_region = NULL;
+  }
+  if (sa) {
+    /* Make overlap active when mouse over. */
+    for (ar = sa->regionbase.first; ar; ar = ar->next) {
+      if (ED_region_contains_xy(ar, xy)) {
+        scr->active_region = ar;
+        break;
+      }
     }
+  }
+  else {
+    scr->active_region = NULL;
+  }
 
-    /* check for redraw headers */
-    if (old_ar != scr->active_region) {
+  /* Check for redraw headers. */
+  if (old_ar != scr->active_region) {
 
-      ED_screen_areas_iter(win, scr, area_iter)
-      {
-        bool do_draw = false;
+    ED_screen_areas_iter(win, scr, area_iter)
+    {
+      bool do_draw = false;
 
-        for (ar = area_iter->regionbase.first; ar; ar = ar->next) {
+      for (ar = area_iter->regionbase.first; ar; ar = ar->next) {
 
-          /* call old area's deactivate if assigned */
-          if (ar == old_ar && area_iter->type->deactivate) {
-            area_iter->type->deactivate(area_iter);
-          }
+        /* Call old area's deactivate if assigned. */
+        if (ar == old_ar && area_iter->type->deactivate) {
+          area_iter->type->deactivate(area_iter);
+        }
 
-          if (ar == old_ar && ar != scr->active_region) {
-            wmGizmoMap *gzmap = old_ar->gizmo_map;
-            if (gzmap) {
-              if (WM_gizmo_highlight_set(gzmap, NULL)) {
-                ED_region_tag_redraw_no_rebuild(old_ar);
-              }
+        if (ar == old_ar && ar != scr->active_region) {
+          wmGizmoMap *gzmap = old_ar->gizmo_map;
+          if (gzmap) {
+            if (WM_gizmo_highlight_set(gzmap, NULL)) {
+              ED_region_tag_redraw_no_rebuild(old_ar);
             }
           }
+        }
 
-          if (ar == old_ar || ar == scr->active_region) {
-            do_draw = true;
-          }
+        if (ar == old_ar || ar == scr->active_region) {
+          do_draw = true;
         }
+      }
 
-        if (do_draw) {
-          for (ar = area_iter->regionbase.first; ar; ar = ar->next) {
-            if (ELEM(ar->regiontype, RGN_TYPE_HEADER, RGN_TYPE_TOOL_HEADER)) {
-              ED_region_tag_redraw_no_rebuild(ar);
-            }
+      if (do_draw) {
+        for (ar = area_iter->regionbase.first; ar; ar = ar->next) {
+          if (ELEM(ar->regiontype, RGN_TYPE_HEADER, RGN_TYPE_TOOL_HEADER)) {
+            ED_region_tag_redraw_no_rebuild(ar);
           }
         }
       }
     }
+  }
 
-    /* cursors, for time being set always on edges, otherwise aregion doesn't switch */
-    if (scr->active_region == NULL) {
-      screen_cursor_set(win, xy);
-    }
-    else {
-      /* notifier invokes freeing the buttons... causing a bit too much redraws */
-      const bool swin_changed = (old_ar != scr->active_region);
-      region_cursor_set_ex(win, sa, scr->active_region, swin_changed);
-
-      if (old_ar != scr->active_region) {
-        /* this used to be a notifier, but needs to be done immediate
-         * because it can undo setting the right button as active due
-         * to delayed notifier handling */
-        if (C) {
-          UI_screen_free_active_but(C, scr);
-        }
+  /* Cursors, for time being set always on edges,
+   * otherwise the active region doesn't switch. */
+  if (scr->active_region == NULL) {
+    screen_cursor_set(win, xy);
+  }
+  else {
+    /* Notifier invokes freeing the buttons... causing a bit too much redraws. */
+    region_cursor_set_ex(win, sa, scr->active_region, old_ar != scr->active_region);
+
+    if (old_ar != scr->active_region) {
+      /* This used to be a notifier, but needs to be done immediate
+       * because it can undo setting the right button as active due
+       * to delayed notifier handling. */
+      if (C) {
+        UI_screen_free_active_but(C, scr);
       }
     }
   }
diff --git a/source/blender/windowmanager/intern/wm_toolsystem.c b/source/blender/windowmanager/intern/wm_toolsystem.c
index 24e9aa0028c..9184a2b4566 100644
--- a/source/blender/windowmanager/intern/wm_toolsystem.c
+++ b/source/blender/windowmanager/intern/wm_toolsystem.c
@@ -661,10 +661,6 @@ static void toolsystem_refresh_screen_from_active_tool(Main *bmain,
             if (mode == tref->mode) {
               sa->runtime.tool = tref;
               sa->runtime.is_tool_set = true;
-              /* We only need to do this for the active area,
-               * however at this point the context isn't available.
-               * Set this here since it doesn't cause any extra work. */
-              sa->flag |= AREA_FLAG_CURSOR_UPDATE;
             }
           }
         }



More information about the Bf-blender-cvs mailing list