[Bf-blender-cvs] [511347ff30b] HMD_viewport: Remove unnecessary check

Julian Eisel noreply at git.blender.org
Tue Apr 11 23:42:29 CEST 2017


Commit: 511347ff30b4fbadf943f5791a56fe30abe19209
Author: Julian Eisel
Date:   Tue Apr 11 23:41:48 2017 +0200
Branches: HMD_viewport
https://developer.blender.org/rB511347ff30b4fbadf943f5791a56fe30abe19209

Remove unnecessary check

Avoids having to pass around window-manager

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

M	source/blender/editors/curve/editcurve_paint.c
M	source/blender/editors/gpencil/gpencil_edit.c
M	source/blender/editors/gpencil/gpencil_paint.c
M	source/blender/editors/gpencil/gpencil_utils.c
M	source/blender/editors/interface/interface_eyedropper.c
M	source/blender/editors/screen/area.c
M	source/blender/editors/sculpt_paint/paint_image_proj.c
M	source/blender/editors/space_view3d/view3d_edit.c
M	source/blender/editors/space_view3d/view3d_view.c
M	source/blender/windowmanager/WM_api.h
M	source/blender/windowmanager/intern/wm_draw.c
M	source/blender/windowmanager/intern/wm_event_system.c
M	source/blender/windowmanager/intern/wm_window.c

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

diff --git a/source/blender/editors/curve/editcurve_paint.c b/source/blender/editors/curve/editcurve_paint.c
index a6015550e97..6243149913d 100644
--- a/source/blender/editors/curve/editcurve_paint.c
+++ b/source/blender/editors/curve/editcurve_paint.c
@@ -1162,7 +1162,7 @@ static int curve_draw_invoke(bContext *C, wmOperator *op, const wmEvent *event)
 			if ((cps->depth_mode == CURVE_PAINT_PROJECT_SURFACE) &&
 			    (v3d->drawtype > OB_WIRE))
 			{
-				const bool is_hmd_view = WM_window_is_hmd_view(CTX_wm_manager(C), CTX_wm_window(C));
+				const bool is_hmd_view = WM_window_is_hmd_view(CTX_wm_window(C));
 
 				view3d_get_transformation(cdd->vc.ar, cdd->vc.rv3d, NULL, &cdd->mats);
 
diff --git a/source/blender/editors/gpencil/gpencil_edit.c b/source/blender/editors/gpencil/gpencil_edit.c
index 90c89e9c250..0fa561ca3de 100644
--- a/source/blender/editors/gpencil/gpencil_edit.c
+++ b/source/blender/editors/gpencil/gpencil_edit.c
@@ -2010,7 +2010,7 @@ static int gp_strokes_reproject_exec(bContext *C, wmOperator *op)
 	
 	/* init autodist for geometry projection */
 	if (mode == GP_REPROJECT_SURFACE) {
-		const bool is_hmd_view = WM_window_is_hmd_view(CTX_wm_manager(C), CTX_wm_window(C));
+		const bool is_hmd_view = WM_window_is_hmd_view(CTX_wm_window(C));
 
 		view3d_region_operator_needs_opengl(CTX_wm_window(C), gsc.ar);
 		ED_view3d_autodist_init(scene, gsc.ar, CTX_wm_view3d(C), 0, is_hmd_view);
diff --git a/source/blender/editors/gpencil/gpencil_paint.c b/source/blender/editors/gpencil/gpencil_paint.c
index 8e890d97342..83d60c7e509 100644
--- a/source/blender/editors/gpencil/gpencil_paint.c
+++ b/source/blender/editors/gpencil/gpencil_paint.c
@@ -114,7 +114,6 @@ typedef enum eGPencil_PaintFlags {
 typedef struct tGPsdata {
 	Scene *scene;       /* current scene from context */
 
-	wmWindowManager *wm; /* window manager where painting originated */
 	wmWindow *win;      /* window where painting originated */
 	ScrArea *sa;        /* area where painting originated */
 	ARegion *ar;        /* region where painting originated */
@@ -635,7 +634,7 @@ static short gp_stroke_addpoint(tGPsdata *p, const int mval[2], float pressure,
 			 */
 			if (gpencil_project_check(p)) {
 				View3D *v3d = p->sa->spacedata.first;
-				const bool is_hmd_view = WM_window_is_hmd_view(p->wm, p->win);
+				const bool is_hmd_view = WM_window_is_hmd_view(p->win);
 				
 				view3d_region_operator_needs_opengl(p->win, p->ar);
 				ED_view3d_autodist_init(p->scene, p->ar, v3d,
@@ -1240,7 +1239,7 @@ static void gp_stroke_doeraser(tGPsdata *p)
 	if (p->sa->spacetype == SPACE_VIEW3D) {
 		if (p->flags & GP_PAINTFLAG_V3D_ERASER_DEPTH) {
 			View3D *v3d = p->sa->spacedata.first;
-			const bool is_hmd_view = WM_window_is_hmd_view(p->wm, p->win);
+			const bool is_hmd_view = WM_window_is_hmd_view(p->win);
 			
 			view3d_region_operator_needs_opengl(p->win, p->ar);
 			ED_view3d_autodist_init(p->scene, p->ar, v3d, 0, is_hmd_view);
@@ -1397,7 +1396,6 @@ static bool gp_session_initdata(bContext *C, tGPsdata *p)
 	
 	/* pass on context info */
 	p->scene = CTX_data_scene(C);
-	p->wm = CTX_wm_manager(C);
 	p->win = CTX_wm_window(C);
 	
 	unit_m4(p->imat);
@@ -1805,7 +1803,7 @@ static void gp_paint_strokeend(tGPsdata *p)
 	 */
 	if (gpencil_project_check(p)) {
 		View3D *v3d = p->sa->spacedata.first;
-		const bool is_hmd_view = WM_window_is_hmd_view(p->wm, p->win);
+		const bool is_hmd_view = WM_window_is_hmd_view(p->win);
 		
 		/* need to restore the original projection settings before packing up */
 		view3d_region_operator_needs_opengl(p->win, p->ar);
diff --git a/source/blender/editors/gpencil/gpencil_utils.c b/source/blender/editors/gpencil/gpencil_utils.c
index fa726e98f50..1ad9d4009fc 100644
--- a/source/blender/editors/gpencil/gpencil_utils.c
+++ b/source/blender/editors/gpencil/gpencil_utils.c
@@ -523,7 +523,7 @@ void gp_point_conversion_init(bContext *C, GP_SpaceConversion *r_gsc)
 		Scene *scene = CTX_data_scene(C);
 		View3D *v3d = (View3D *)CTX_wm_space_data(C);
 		RegionView3D *rv3d = ar->regiondata;
-		const bool is_hmd_view = WM_window_is_hmd_view(CTX_wm_manager(C), CTX_wm_window(C));
+		const bool is_hmd_view = WM_window_is_hmd_view(CTX_wm_window(C));
 		
 		/* init 3d depth buffers */
 		view3d_operator_needs_opengl(C);
diff --git a/source/blender/editors/interface/interface_eyedropper.c b/source/blender/editors/interface/interface_eyedropper.c
index 9d00ffa3c40..003490a3181 100644
--- a/source/blender/editors/interface/interface_eyedropper.c
+++ b/source/blender/editors/interface/interface_eyedropper.c
@@ -888,7 +888,7 @@ static void depthdropper_depth_sample_pt(bContext *C, DepthDropper *ddr, int mx,
 	Scene *scene = win->screen->scene;
 	UnitSettings *unit = &scene->unit;
 	const bool do_split = (unit->flag & USER_UNIT_OPT_SPLIT) != 0;
-	const bool is_hmd_view = WM_window_is_hmd_view(CTX_wm_manager(C), win);
+	const bool is_hmd_view = WM_window_is_hmd_view(win);
 
 	ScrArea *area_prev = CTX_wm_area(C);
 	ARegion *ar_prev = CTX_wm_region(C);
diff --git a/source/blender/editors/screen/area.c b/source/blender/editors/screen/area.c
index 35c97ff070c..601debe7e89 100644
--- a/source/blender/editors/screen/area.c
+++ b/source/blender/editors/screen/area.c
@@ -473,13 +473,13 @@ void ED_region_set(const bContext *C, ARegion *ar)
 	ED_region_pixelspace(ar);
 }
 
-static void region_draw_view_setup(wmWindowManager *wm, wmWindow *win, ARegion *ar)
+static void region_draw_view_setup(wmWindow *win, ARegion *ar)
 {
 #ifdef WITH_INPUT_HMD
 	if (ar->regiontype == RGN_TYPE_TEMPORARY) {
 		/* pass */
 	}
-	else if (!WM_window_is_hmd_view(wm, win)) {
+	else if (!WM_window_is_hmd_view(win)) {
 		/* pass */
 	}
 	else {
@@ -488,16 +488,16 @@ static void region_draw_view_setup(wmWindowManager *wm, wmWindow *win, ARegion *
 		wm_subwindow_rect_set(win, ar->swinid, &ar->winrct);
 	}
 #else
-	UNUSED_VARS(wm, win, ar);
+	UNUSED_VARS(win, ar);
 #endif
 }
-static void region_draw_view_reset(wmWindowManager *wm, wmWindow *win, ARegion *ar)
+static void region_draw_view_reset(wmWindow *win, ARegion *ar)
 {
 #ifdef WITH_INPUT_HMD
 	if (ar->regiontype == RGN_TYPE_TEMPORARY) {
 		/* pass */
 	}
-	else if (!WM_window_is_hmd_view(wm, win)) {
+	else if (!WM_window_is_hmd_view(win)) {
 		/* pass */
 	}
 	else {
@@ -506,14 +506,13 @@ static void region_draw_view_reset(wmWindowManager *wm, wmWindow *win, ARegion *
 		wm_subwindow_rect_set(win, ar->swinid, &ar->winrct);
 	}
 #else
-	UNUSED_VARS(wm, win, ar);
+	UNUSED_VARS(win, ar);
 #endif
 }
 
 /* only exported for WM */
 void ED_region_do_draw(bContext *C, ARegion *ar)
 {
-	wmWindowManager *wm = CTX_wm_manager(C);
 	wmWindow *win = CTX_wm_window(C);
 	ScrArea *sa = CTX_wm_area(C);
 	ARegionType *at = ar->type;
@@ -523,7 +522,7 @@ void ED_region_do_draw(bContext *C, ARegion *ar)
 	if (at->do_lock)
 		return;
 
-	region_draw_view_setup(wm, win, ar);
+	region_draw_view_setup(win, ar);
 
 	/* if no partial draw rect set, full rect */
 	if (ar->drawrct.xmin == ar->drawrct.xmax) {
@@ -585,7 +584,7 @@ void ED_region_do_draw(bContext *C, ARegion *ar)
 		}
 	}
 
-	region_draw_view_reset(wm, win, ar);
+	region_draw_view_reset(win, ar);
 }
 
 /* **********************************
diff --git a/source/blender/editors/sculpt_paint/paint_image_proj.c b/source/blender/editors/sculpt_paint/paint_image_proj.c
index 04ef1774663..af564f733cc 100644
--- a/source/blender/editors/sculpt_paint/paint_image_proj.c
+++ b/source/blender/editors/sculpt_paint/paint_image_proj.c
@@ -5021,7 +5021,7 @@ void paint_proj_stroke(
 		ARegion *ar = CTX_wm_region(C);
 		float *cursor = ED_view3d_cursor3d_get(scene, v3d);
 		int mval_i[2] = {(int)pos[0], (int)pos[1]};
-		const bool is_hmd_view = WM_window_is_hmd_view(CTX_wm_manager(C), CTX_wm_window(C));
+		const bool is_hmd_view = WM_window_is_hmd_view(CTX_wm_window(C));
 
 		view3d_operator_needs_opengl(C);
 
diff --git a/source/blender/editors/space_view3d/view3d_edit.c b/source/blender/editors/space_view3d/view3d_edit.c
index 01119bc7d33..3aa04ee56f4 100644
--- a/source/blender/editors/space_view3d/view3d_edit.c
+++ b/source/blender/editors/space_view3d/view3d_edit.c
@@ -733,7 +733,7 @@ static void viewops_data_create_ex(bContext *C, wmOperator *op, const wmEvent *e
 	else if (use_orbit_zbuf) {
 		Scene *scene = CTX_data_scene(C);
 		float fallback_depth_pt[3];
-		const bool is_hmd_view = WM_window_is_hmd_view(CTX_wm_manager(C), CTX_wm_window(C));
+		const bool is_hmd_view = WM_window_is_hmd_view(CTX_wm_window(C));
 
 		view3d_operator_needs_opengl(C); /* needed for zbuf drawing */
 
@@ -3279,7 +3279,7 @@ static int viewcenter_pick_invoke(bContext *C, wmOperator *op, const wmEvent *ev
 	if (rv3d) {
 		float new_ofs[3];
 		const int smooth_viewtx = WM_operator_smooth_viewtx_get(op);
-		const bool is_hmd_view = WM_window_is_hmd_view(CTX_wm_manager(C), CTX_wm_window(C));
+		const bool is_hmd_view = WM_window_is_hmd_view(CTX_wm_window(C));
 
 		ED_view3d_smooth_view_force_finish(C, v3d, ar);
 
@@ -3546,7 +3546,7 @@ static int view3d_zoom_border_exec(bContext *C, wmOperator *op)
 	Scene *scene = CTX_data_scene(C);
 	int gesture_mode;
 	const int smooth_viewtx = WM_operator_smooth_viewtx_get(op);
-	const bool is_hmd_view = WM_window_is_hmd_view(CTX_wm_manager(C), CTX_wm_window(C));
+	const bool is_hmd_view = WM_window_is_hmd_view(CTX_wm_window(C));
 
 	/* Zooms in on a border drawn by the user */
 	rcti rect;
@@ -4701,7 +4701,7 @@ void ED_view3d_cursor3d_position(bContext *C, float fp[3], const int mval[2])
 	}
 
 	if (U.uiflag & USER_ZBUF_CURSOR) {  /* maybe this should be accessed some other way */
-		const bool is_hmd_view = WM_window_is_hmd_view(CTX_wm_manager(C), CTX_wm_window(C));
+		const bool is_hmd_view = WM_window_is_hmd_view(CTX_wm_window(C));
 
 		view3d_operator_needs_opengl(C);
 		if (ED_view3d_autodist(scene, ar, v3d, mval, fp, true, is_hmd_view, NULL)) {
diff --git a/source/blender/editors/space_view3d/view3d_view.c b/source/blender/editors/space_view3d/view3d_view.c
index d8883179b09..16233e054fc 100644
--- a/source/blender/editors/space_view3d/view3d_view.c
+++ b/source/blender/editors/space_view3d/view3d_view.c
@@ -1197,7 +1197,7 @@ int view3d_opengl_select(
 
 #ifdef WITH_INPUT_HMD
 	wmWindowManager *wm = G.main->wm.fi

@@ Diff output truncated at 10240 characters. @@




More information about the Bf-blender-cvs mailing list