[Bf-blender-cvs] [f1bc8991461] blender2.8: Merge branch 'master' into blender2.8

Campbell Barton noreply at git.blender.org
Tue May 15 10:39:02 CEST 2018


Commit: f1bc8991461fcda33eb34820bd3aa153f6122c5b
Author: Campbell Barton
Date:   Tue May 15 10:25:49 2018 +0200
Branches: blender2.8
https://developer.blender.org/rBf1bc8991461fcda33eb34820bd3aa153f6122c5b

Merge branch 'master' into blender2.8

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



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

diff --cc source/blender/editors/animation/anim_channels_edit.c
index 36402b82cbf,301124c4dfa..0163a7f68d6
--- a/source/blender/editors/animation/anim_channels_edit.c
+++ b/source/blender/editors/animation/anim_channels_edit.c
@@@ -2725,8 -2720,9 +2725,9 @@@ static int mouse_anim_channels(bContex
  					adt->flag |= ADT_UI_ACTIVE;
  
  				/* ensure we exit editmode on whatever object was active before to avoid getting stuck there - T48747 */
- 				if (ob != CTX_data_edit_object(C))
- 					ED_object_editmode_exit(C, EM_FREEDATA | EM_WAITCURSOR | EM_DO_UNDO);
 -				if (ob != sce->obedit) {
++				if (ob != CTX_data_edit_object(C)) {
+ 					ED_object_editmode_exit(C, EM_FREEDATA | EM_WAITCURSOR);
+ 				}
  
  				notifierFlags |= (ND_ANIMCHAN | NA_SELECTED);
  			}
diff --cc source/blender/editors/include/ED_object.h
index 0775c2b953b,614be5dd0c1..8e844d327e2
--- a/source/blender/editors/include/ED_object.h
+++ b/source/blender/editors/include/ED_object.h
@@@ -122,15 -115,10 +122,14 @@@ void ED_object_parent(struct Object *ob
  enum {
  	EM_FREEDATA         = (1 << 0),
  	EM_WAITCURSOR       = (1 << 1),
- 	EM_DO_UNDO          = (1 << 2),
  	EM_IGNORE_LAYER     = (1 << 3),
 +	EM_NO_CONTEXT       = (1 << 4),
  };
 -void ED_object_editmode_exit_ex(struct Scene *scene, struct Object *obedit, int flag);
 +void ED_object_editmode_exit_ex(
-         struct bContext *C, struct Scene *scene, struct Object *obedit, int flag);
++        struct Scene *scene, struct Object *obedit, int flag);
  void ED_object_editmode_exit(struct bContext *C, int flag);
 +
 +void ED_object_editmode_enter_ex(struct Scene *scene, struct Object *ob, int flag);
  void ED_object_editmode_enter(struct bContext *C, int flag);
  bool ED_object_editmode_load(struct Object *obedit);
  
diff --cc source/blender/editors/mesh/editmesh_add.c
index 292b28c772c,9bea203e67b..66ad057e3ec
--- a/source/blender/editors/mesh/editmesh_add.c
+++ b/source/blender/editors/mesh/editmesh_add.c
@@@ -77,11 -72,11 +77,11 @@@ static Object *make_prim_init
  		obedit = ED_object_add_type(C, OB_MESH, idname, loc, rot, false, layer);
  
  		/* create editmode */
- 		ED_object_editmode_enter(C, EM_DO_UNDO | EM_IGNORE_LAYER); /* rare cases the active layer is messed up */
+ 		ED_object_editmode_enter(C, EM_IGNORE_LAYER); /* rare cases the active layer is messed up */
 -		*was_editmode = true;
 +		r_creation_data->was_editmode = true;
  	}
  
 -	*dia = ED_object_new_primitive_matrix(C, obedit, loc, rot, mat);
 +	ED_object_new_primitive_matrix(C, obedit, loc, rot, r_creation_data->mat);
  
  	return obedit;
  }
diff --cc source/blender/editors/object/object_add.c
index 366667163b0,101b5d4920d..b1c3c8bc59b
--- a/source/blender/editors/object/object_add.c
+++ b/source/blender/editors/object/object_add.c
@@@ -424,24 -411,29 +424,25 @@@ Object *ED_object_add_type
  	Object *ob;
  
  	/* for as long scene has editmode... */
- 	if (CTX_data_edit_object(C)) 
- 		ED_object_editmode_exit(C, EM_FREEDATA | EM_WAITCURSOR | EM_DO_UNDO);  /* freedata, and undo */
+ 	if (CTX_data_edit_object(C)) {
+ 		ED_object_editmode_exit(C, EM_FREEDATA | EM_WAITCURSOR);
+ 	}
  
  	/* deselects all, sets scene->basact */
 -	ob = BKE_object_add(bmain, scene, type, name);
 -	BASACT->lay = ob->lay = layer;
 +	ob = BKE_object_add(bmain, scene, view_layer, type, name);
  	/* editor level activate, notifiers */
 -	ED_base_object_activate(C, BASACT);
 +	ED_object_base_activate(C, view_layer->basact);
  
  	/* more editor stuff */
 -	ED_object_base_init_transform(C, BASACT, loc, rot);
 -
 -	/* Ignore collisions by default for non-mesh objects */
 -	if (type != OB_MESH) {
 -		ob->body_type = OB_BODY_TYPE_NO_COLLISION;
 -		ob->gameflag &= ~(OB_SENSOR | OB_RIGID_BODY | OB_SOFT_BODY | OB_COLLISION | OB_CHARACTER | OB_OCCLUDER | OB_DYNAMIC | OB_NAVMESH); /* copied from rna_object.c */
 -	}
 +	ED_object_base_init_transform(C, view_layer->basact, loc, rot);
  
 -	DAG_id_type_tag(bmain, ID_OB);
 -	DAG_relations_tag_update(bmain);
 -	if (ob->data) {
 -		ED_render_id_flush_update(bmain, ob->data);
 +	/* TODO(sergey): This is weird to manually tag objects for update, better to
 +	 * use DEG_id_tag_update here perhaps.
 +	 */
 +	DEG_id_type_tag(bmain, ID_OB);
 +	DEG_relations_tag_update(bmain);
 +	if (ob->data != NULL) {
 +		DEG_id_tag_update_ex(bmain, (ID *)ob->data, DEG_TAG_EDITORS_UPDATE);
  	}
  
  	if (enter_editmode)
diff --cc source/blender/editors/object/object_edit.c
index 4b13419f425,a01a671ec98..e87a24b3af0
--- a/source/blender/editors/object/object_edit.c
+++ b/source/blender/editors/object/object_edit.c
@@@ -306,19 -482,10 +303,11 @@@ void ED_object_editmode_exit_ex(Scene *
  		BKE_ptcache_object_reset(scene, obedit, PTCACHE_RESET_OUTDATED);
  
  		/* also flush ob recalc, doesn't take much overhead, but used for particles */
 -		DAG_id_tag_update(&obedit->id, OB_RECALC_OB | OB_RECALC_DATA);
 +		DEG_id_tag_update(&obedit->id, OB_RECALC_OB | OB_RECALC_DATA);
- 	
- 		if (flag & EM_DO_UNDO)
- 			ED_undo_push(C, "Editmode");
  
- 		if (C != NULL) {
- 			WM_event_add_notifier(C, NC_SCENE | ND_MODE | NS_MODE_OBJECT, scene);
- 		}
- 		else {
- 			WM_main_add_notifier(NC_SCENE | ND_MODE | NS_MODE_OBJECT, scene);
- 		}
+ 		WM_main_add_notifier(NC_SCENE | ND_MODE | NS_MODE_OBJECT, scene);
  
 +
  		obedit->mode &= ~OB_MODE_EDIT;
  	}
  
@@@ -329,15 -496,37 +318,15 @@@ void ED_object_editmode_exit(bContext *
  {
  	Scene *scene = CTX_data_scene(C);
  	Object *obedit = CTX_data_edit_object(C);
- 	ED_object_editmode_exit_ex(C, scene, obedit, flag);
+ 	ED_object_editmode_exit_ex(scene, obedit, flag);
  }
  
 -void ED_object_editmode_enter(bContext *C, int flag)
 +void ED_object_editmode_enter_ex(Scene *scene, Object *ob, int flag)
  {
 -	Scene *scene = CTX_data_scene(C);
 -	Base *base = NULL;
 -	Object *ob;
 -	ScrArea *sa = CTX_wm_area(C);
 -	View3D *v3d = NULL;
  	bool ok = false;
  
 -	if (ID_IS_LINKED(scene)) return;
 -
 -	if (sa && sa->spacetype == SPACE_VIEW3D)
 -		v3d = sa->spacedata.first;
 -
 -	if ((flag & EM_IGNORE_LAYER) == 0) {
 -		base = CTX_data_active_base(C); /* active layer checked here for view3d */
 -
 -		if (base == NULL) return;
 -		else if (v3d && (base->lay & v3d->lay) == 0) return;
 -		else if (!v3d && (base->lay & scene->lay) == 0) return;
 -	}
 -	else {
 -		base = scene->basact;
 -	}
 -
 -	if (ELEM(NULL, base, base->object, base->object->data)) return;
 -
 -	ob = base->object;
 +	if (ELEM(NULL, ob, ob->data)) return;
 +	if (ID_IS_LINKED(ob)) return;
  
  	/* this checks actual object->data, for cases when other scenes have it in editmode context */
  	if (BKE_object_is_in_editmode(ob))
@@@ -428,31 -628,10 +417,29 @@@
  	}
  
  	if (flag & EM_WAITCURSOR) waitcursor(0);
- 	BLI_assert((flag & EM_DO_UNDO) == 0);
  }
  
 +void ED_object_editmode_enter(bContext *C, int flag)
 +{
 +	Scene *scene = CTX_data_scene(C);
 +	ViewLayer *view_layer = CTX_data_view_layer(C);
 +	Object *ob;
 +
 +	if ((flag & EM_IGNORE_LAYER) == 0) {
 +		ob = CTX_data_active_object(C); /* active layer checked here for view3d */
 +	}
 +	else {
 +		ob = view_layer->basact->object;
 +	}
 +	if (ob == NULL) return;
 +	if (ID_IS_LINKED(ob)) return;
 +
- 	ED_object_editmode_enter_ex(scene, ob, flag & ~EM_DO_UNDO);
- 	if (flag & EM_DO_UNDO) ED_undo_push(C, "Enter Editmode");
++	ED_object_editmode_enter_ex(scene, ob, flag);
 +}
 +
  static int editmode_toggle_exec(bContext *C, wmOperator *op)
  {
 +	struct wmMsgBus *mbus = CTX_wm_message_bus(C);
  	const int mode_flag = OB_MODE_EDIT;
  	const bool is_mode_set = (CTX_data_edit_object(C) != NULL);
  	Scene *scene =  CTX_data_scene(C);
@@@ -467,33 -645,12 +454,33 @@@
  
  	if (!is_mode_set) {
  		ED_object_editmode_enter(C, EM_WAITCURSOR);
 +		if (obact->mode & mode_flag) {
 +			FOREACH_SELECTED_OBJECT_BEGIN(view_layer, ob)
 +			{
 +				if ((ob != obact) && (ob->type == obact->type)) {
 +					ED_object_editmode_enter_ex(scene, ob, EM_WAITCURSOR | EM_NO_CONTEXT);
 +				}
 +			}
 +			FOREACH_SELECTED_OBJECT_END;
 +		}
  	}
  	else {
- 		ED_object_editmode_exit(C, EM_FREEDATA | EM_WAITCURSOR);  /* had EM_DO_UNDO but op flag calls undo too [#24685] */
+ 		ED_object_editmode_exit(C, EM_FREEDATA | EM_WAITCURSOR);
 +		if ((obact->mode & mode_flag) == 0) {
 +			FOREACH_SELECTED_OBJECT_BEGIN(view_layer, ob)
 +			{
 +				if ((ob != obact) && (ob->type == obact->type)) {
- 					ED_object_editmode_exit_ex(NULL, scene, ob, EM_FREEDATA | EM_WAITCURSOR);
++					ED_object_editmode_exit_ex(scene, ob, EM_FREEDATA | EM_WAITCURSOR);
 +				}
 +			}
 +			FOREACH_SELECTED_OBJECT_END;
 +		}
  	}
 +	
  	ED_space_image_uv_sculpt_update(CTX_wm_manager(C), scene);
  
 +	WM_msg_publish_rna_prop(mbus, &obact->id, obact, Object, mode);
 +
  	return OPERATOR_FINISHED;
  }
  
@@@ -545,54 -700,23 +532,54 @@@ static int posemode_exec(bContext *C, w
  		}
  	}
  
 -	if (ob->type == OB_ARMATURE) {
 -		if (ob == CTX_data_edit_object(C)) {
 -			ED_object_editmode_exit(C, EM_FREEDATA);
 -			is_mode_set = false;
 -		}
 +	if (obact->type != OB_ARMATURE) {
 +		return OPERATOR_PASS_THROUGH;
 +	}
 +
 +	if (obact == CTX_data_edit_object(C)) {
- 		ED_object_editmode_exit(C, EM_FREEDATA | EM_DO_UNDO);
++		ED_object_editmode_exit(C, EM_FREEDATA);
 +		is_mode_set = false;
 +	}
  
 -		if (is_mode_set) {
 -			ED_object_posemode_exit(C, ob);
 +	if (is_mode_set) {
 +		bool ok = ED_object_posemode_exit(C, obact);
 +		if (ok) {
 +			struct Main *bmain = CTX_data_main(C);
 +			ViewLayer *view_layer = CTX_data_view_layer(C);
 +			FOREACH_SELECTED_OBJECT_BEGIN(view_layer, ob)
 +			{
 +				if ((ob != obact) &&
 +				    (ob->type == OB_ARMATURE) &&
 +				    (ob->mode & mode_flag))
 +				{
 +					ED_object_posemode_exit_ex(bmain, ob);
 +				}
 +			}
 +			FOREACH_SELECTED_OBJECT_END;
  		}
 -		else {
 -			ED_object_posemode_enter(C, ob);
 +	}
 +	else {
 +		bool ok = ED_object_posemode_enter(C, obact);
 +		if (ok) {
 +			struct Main *bmain = CTX_data_main(C);
 +			ViewLayer *view_layer = CTX_data_view_layer(C);
 +			FOREACH_SELECTED_OBJECT_BEGIN(view_layer, ob)
 +			{
 +				if ((ob != obact) &&
 +				    (ob->type == OB_ARMATURE) &&
 +				    (ob->mode == OB_MODE_OBJECT) &&
 +				    (!ID_IS_LINKED(ob)))
 +				{
 +					ED_object_posemode_enter_ex(bmain, ob);
 +				}
 +			}
 +			FOREACH_SELECTED_O

@@ Diff output truncated at 10240 characters. @@



More information about the Bf-blender-cvs mailing list