[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [46031] trunk/blender/source/blender: code cleanup: replace some long ELEM7 checks with macro OB_TYPE_SUPPORT_EDITMODE()

Campbell Barton ideasman42 at gmail.com
Sat Apr 28 12:33:33 CEST 2012


Revision: 46031
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=46031
Author:   campbellbarton
Date:     2012-04-28 10:33:32 +0000 (Sat, 28 Apr 2012)
Log Message:
-----------
code cleanup: replace some long ELEM7 checks with macro OB_TYPE_SUPPORT_EDITMODE()

Modified Paths:
--------------
    trunk/blender/source/blender/editors/object/object_edit.c
    trunk/blender/source/blender/editors/space_view3d/view3d_header.c
    trunk/blender/source/blender/editors/util/undo.c
    trunk/blender/source/blender/makesdna/DNA_object_types.h

Modified: trunk/blender/source/blender/editors/object/object_edit.c
===================================================================
--- trunk/blender/source/blender/editors/object/object_edit.c	2012-04-28 10:09:58 UTC (rev 46030)
+++ trunk/blender/source/blender/editors/object/object_edit.c	2012-04-28 10:33:32 UTC (rev 46031)
@@ -1351,11 +1351,11 @@
 	if (!C) /* needed for docs */
 		return object_mode_items;
 
+
+
 	ob = CTX_data_active_object(C);
 	while (ob && input->identifier) {
-		if ((input->value == OB_MODE_EDIT && ((ob->type == OB_MESH) || (ob->type == OB_ARMATURE) ||
-		                                      (ob->type == OB_CURVE) || (ob->type == OB_SURF) ||
-		                                      (ob->type == OB_FONT) || (ob->type == OB_MBALL) || (ob->type == OB_LATTICE))) ||
+		if ((input->value == OB_MODE_EDIT && OB_TYPE_SUPPORT_EDITMODE(ob->type)) ||
 		    (input->value == OB_MODE_POSE && (ob->type == OB_ARMATURE)) ||
 		    (input->value == OB_MODE_PARTICLE_EDIT && ob->particlesystem.first) ||
 		    ((input->value == OB_MODE_SCULPT || input->value == OB_MODE_VERTEX_PAINT ||

Modified: trunk/blender/source/blender/editors/space_view3d/view3d_header.c
===================================================================
--- trunk/blender/source/blender/editors/space_view3d/view3d_header.c	2012-04-28 10:09:58 UTC (rev 46030)
+++ trunk/blender/source/blender/editors/space_view3d/view3d_header.c	2012-04-28 10:33:32 UTC (rev 46031)
@@ -294,15 +294,15 @@
 
 	if (!((ID *)ob->data)->lib) {
 		/* if active object is editable */
-		if (ELEM6(ob->type, OB_MESH, OB_CURVE, OB_SURF, OB_FONT, OB_MBALL, OB_LATTICE)) {
-			str += modeselect_addmode(str, N_("Edit Mode"), OB_MODE_EDIT, ICON_EDITMODE_HLT);
-		}
-		else if (ob->type == OB_ARMATURE) {
+		if (ob->type == OB_ARMATURE) {
 			if (ob->mode & OB_MODE_POSE)
 				str += modeselect_addmode(str, N_("Edit Mode"), OB_MODE_EDIT | OB_MODE_POSE, ICON_EDITMODE_HLT);
 			else
 				str += modeselect_addmode(str, N_("Edit Mode"), OB_MODE_EDIT, ICON_EDITMODE_HLT);
 		}
+		else if (OB_TYPE_SUPPORT_EDITMODE(ob->type)) {
+			str += modeselect_addmode(str, N_("Edit Mode"), OB_MODE_EDIT, ICON_EDITMODE_HLT);
+		}
 
 		if (ob->type == OB_MESH) {
 

Modified: trunk/blender/source/blender/editors/util/undo.c
===================================================================
--- trunk/blender/source/blender/editors/util/undo.c	2012-04-28 10:09:58 UTC (rev 46030)
+++ trunk/blender/source/blender/editors/util/undo.c	2012-04-28 10:33:32 UTC (rev 46031)
@@ -155,7 +155,7 @@
 		ED_text_undo_step(C, step);
 	}
 	else if (obedit) {
-		if (ELEM7(obedit->type, OB_MESH, OB_FONT, OB_CURVE, OB_SURF, OB_MBALL, OB_LATTICE, OB_ARMATURE)) {
+		if (OB_TYPE_SUPPORT_EDITMODE(obedit->type)) {
 			if (undoname)
 				undo_editmode_name(C, undoname);
 			else
@@ -247,7 +247,7 @@
 		return 1;
 	}
 	else if (obedit) {
-		if (ELEM7(obedit->type, OB_MESH, OB_FONT, OB_CURVE, OB_SURF, OB_MBALL, OB_LATTICE, OB_ARMATURE)) {
+		if (OB_TYPE_SUPPORT_EDITMODE(obedit->type)) {
 			return undo_editmode_valid(undoname);
 		}
 	}
@@ -413,9 +413,11 @@
 
 /* ************************** */
 
-#define UNDOSYSTEM_GLOBAL   1
-#define UNDOSYSTEM_EDITMODE 2
-#define UNDOSYSTEM_PARTICLE 3
+enum {
+	UNDOSYSTEM_GLOBAL   = 1,
+	UNDOSYSTEM_EDITMODE = 2,
+	UNDOSYSTEM_PARTICLE = 3
+};
 
 static int get_undo_system(bContext *C)
 {
@@ -423,8 +425,9 @@
 	
 	/* find out which undo system */
 	if (obedit) {
-		if (ELEM7(obedit->type, OB_MESH, OB_FONT, OB_CURVE, OB_SURF, OB_MBALL, OB_LATTICE, OB_ARMATURE))
+		if (OB_TYPE_SUPPORT_EDITMODE(obedit->type)) {
 			return UNDOSYSTEM_EDITMODE;
+		}
 	}
 	else {
 		Object *obact = CTX_data_active_object(C);

Modified: trunk/blender/source/blender/makesdna/DNA_object_types.h
===================================================================
--- trunk/blender/source/blender/makesdna/DNA_object_types.h	2012-04-28 10:09:58 UTC (rev 46030)
+++ trunk/blender/source/blender/makesdna/DNA_object_types.h	2012-04-28 10:33:32 UTC (rev 46031)
@@ -319,8 +319,12 @@
 #define	OB_ARMATURE		25
 
 /* check if the object type supports materials */
-#define OB_TYPE_SUPPORT_MATERIAL(_type) ((_type)  >= OB_MESH && (_type) <= OB_MBALL)
-#define OB_TYPE_SUPPORT_VGROUP(_type)   (ELEM(_type, OB_MESH, OB_LATTICE))
+#define OB_TYPE_SUPPORT_MATERIAL(_type) \
+	((_type)  >= OB_MESH && (_type) <= OB_MBALL)
+#define OB_TYPE_SUPPORT_VGROUP(_type) \
+	(ELEM(_type, OB_MESH, OB_LATTICE))
+#define OB_TYPE_SUPPORT_EDITMODE(_type) \
+	(ELEM7(_type, OB_MESH, OB_FONT, OB_CURVE, OB_SURF, OB_MBALL, OB_LATTICE, OB_ARMATURE))
 
 /* partype: first 4 bits: type */
 #define PARTYPE			15




More information about the Bf-blender-cvs mailing list