Merge branch 'master' into blender2.8
[blender.git] / source / blender / editors / object / object_edit.c
index 57d929ceccba59b7548cbf3dc8b69f7c4cc3d30d..65fe578a6025d17ee44765058630b448e28f051f 100644 (file)
@@ -272,7 +272,7 @@ bool ED_object_editmode_load(Object *obedit)
  * \param flag:
  * - If #EM_FREEDATA isn't in the flag, use ED_object_editmode_load directly.
  */
-void ED_object_editmode_exit_ex(Scene *scene, Object *obedit, int flag)
+bool ED_object_editmode_exit_ex(Scene *scene, Object *obedit, int flag)
 {
        const bool freedata = (flag & EM_FREEDATA) != 0;
 
@@ -285,7 +285,7 @@ void ED_object_editmode_exit_ex(Scene *scene, Object *obedit, int flag)
                        obedit->mode &= ~OB_MODE_EDIT;
                }
                if (flag & EM_WAITCURSOR) waitcursor(0);
-               return;
+               return true;
        }
 
        /* freedata only 0 now on file saves and render */
@@ -300,7 +300,7 @@ void ED_object_editmode_exit_ex(Scene *scene, Object *obedit, int flag)
                                pid->cache->flag |= PTCACHE_OUTDATED;
                }
                BLI_freelistN(&pidlist);
-               
+
                BKE_ptcache_object_reset(scene, obedit, PTCACHE_RESET_OUTDATED);
 
                /* also flush ob recalc, doesn't take much overhead, but used for particles */
@@ -308,34 +308,37 @@ void ED_object_editmode_exit_ex(Scene *scene, Object *obedit, int flag)
 
                WM_main_add_notifier(NC_SCENE | ND_MODE | NS_MODE_OBJECT, scene);
 
-
                obedit->mode &= ~OB_MODE_EDIT;
        }
 
        if (flag & EM_WAITCURSOR) waitcursor(0);
+
+       return (obedit->mode & OB_MODE_EDIT) == 0;
 }
 
-void ED_object_editmode_exit(bContext *C, int flag)
+bool ED_object_editmode_exit(bContext *C, int flag)
 {
        Scene *scene = CTX_data_scene(C);
        Object *obedit = CTX_data_edit_object(C);
-       ED_object_editmode_exit_ex(scene, obedit, flag);
+       return ED_object_editmode_exit_ex(scene, obedit, flag);
 }
 
-void ED_object_editmode_enter_ex(Scene *scene, Object *ob, int flag)
+bool ED_object_editmode_enter_ex(Scene *scene, Object *ob, int flag)
 {
        bool ok = false;
 
-       if (ELEM(NULL, ob, ob->data)) return;
-       if (ID_IS_LINKED(ob)) return;
+       if (ELEM(NULL, ob, ob->data) || ID_IS_LINKED(ob)) {
+               return false;
+       }
 
        /* this checks actual object->data, for cases when other scenes have it in editmode context */
-       if (BKE_object_is_in_editmode(ob))
-               return;
-       
+       if (BKE_object_is_in_editmode(ob)) {
+               return true;
+       }
+
        if (BKE_object_obdata_is_libdata(ob)) {
                error_libdata();
-               return;
+               return false;
        }
 
        if (flag & EM_WAITCURSOR) waitcursor(1);
@@ -361,22 +364,8 @@ void ED_object_editmode_enter_ex(Scene *scene, Object *ob, int flag)
                WM_main_add_notifier(NC_SCENE | ND_MODE | NS_EDITMODE_MESH, NULL);
        }
        else if (ob->type == OB_ARMATURE) {
-               bArmature *arm = ob->data;
-               if (!arm) return;
-               /*
-                * The function BKE_object_obdata_is_libdata make a problem here, the
-                * check for ob->proxy return 0 and let blender enter to edit mode
-                * this causes a crash when you try leave the edit mode.
-                * The problem is that i can't remove the ob->proxy check from
-                * BKE_object_obdata_is_libdata that prevent the bugfix #6614, so
-                * i add this little hack here.
-                */
-               if (ID_IS_LINKED(arm)) {
-                       error_libdata();
-                       return;
-               }
                ok = 1;
-               ED_armature_to_edit(arm);
+               ED_armature_to_edit(ob->data);
                /* to ensure all goes in restposition and without striding */
                DEG_id_tag_update(&ob->id, OB_RECALC_OB | OB_RECALC_DATA | OB_RECALC_TIME); /* XXX: should this be OB_RECALC_DATA? */
 
@@ -418,9 +407,11 @@ void ED_object_editmode_enter_ex(Scene *scene, Object *ob, int flag)
        }
 
        if (flag & EM_WAITCURSOR) waitcursor(0);
+
+       return (ob->mode & OB_MODE_EDIT) != 0;
 }
 
-void ED_object_editmode_enter(bContext *C, int flag)
+bool ED_object_editmode_enter(bContext *C, int flag)
 {
        Scene *scene = CTX_data_scene(C);
        ViewLayer *view_layer = CTX_data_view_layer(C);
@@ -432,10 +423,10 @@ void ED_object_editmode_enter(bContext *C, int flag)
        else {
                ob = view_layer->basact->object;
        }
-       if (ob == NULL) return;
-       if (ID_IS_LINKED(ob)) return;
-
-       ED_object_editmode_enter_ex(scene, ob, flag);
+       if ((ob == NULL) || ID_IS_LINKED(ob)) {
+               return false;
+       }
+       return ED_object_editmode_enter_ex(scene, ob, flag);
 }
 
 static int editmode_toggle_exec(bContext *C, wmOperator *op)