[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [45804] trunk/blender/source/blender/ editors/mesh/editmesh_knife.c: knife tool now selects newly cut edges.

[Campbell Barton]

Revision: 45804
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=45804
Author:   campbellbarton
Date:     2012-04-20 11:14:16 +0000 (Fri, 20 Apr 2012)
Log Message:
-----------
knife tool now selects newly cut edges.

Modified Paths:
--------------
    trunk/blender/source/blender/editors/mesh/editmesh_knife.c

Modified: trunk/blender/source/blender/editors/mesh/editmesh_knife.c
===================================================================
--- trunk/blender/source/blender/editors/mesh/editmesh_knife.c	2012-04-20 11:08:58 UTC (rev 45803)
+++ trunk/blender/source/blender/editors/mesh/editmesh_knife.c	2012-04-20 11:14:16 UTC (rev 45804)
@@ -163,6 +163,7 @@
 	/* operatpr options */
 	char  cut_through; /* preference, can be modified at runtime (that feature may go) */
 	char  only_select; /* set on initialization */
+	char  select_result; /* set on initialization */
 
 	float clipsta, clipend;
 
@@ -2452,7 +2453,8 @@
 		/* Want to prevent creating two-sided polygons */
 		if (BM_edge_exists(v1, v2)) {
 			*newface = NULL;
-		} else {
+		}
+		else {
 			*newface = BM_face_split(bm, f, v1, v2, &lnew, NULL, TRUE);
 		}
 	}
@@ -2464,11 +2466,21 @@
 			/* Now go through lnew chain matching up chain kv's and assign real v's to them */
 			for (l_iter = lnew->next, i = 0; i < nco; l_iter = l_iter->next, i++) {
 				BLI_assert(equals_v3v3(cos[i], l_iter->v->co));
+				if (kcd->select_result) {
+					BM_edge_select_set(bm, l_iter->e, TRUE);
+				}
 				kverts[i]->v = l_iter->v;
 			}
 		}
 	}
 
+	/* the select chain above doesnt account for the first loop */
+	if (kcd->select_result) {
+		if (lnew) {
+			BM_edge_select_set(bm, lnew->e, TRUE);
+		}
+	}
+
 	BLI_array_fixedstack_free(cos);
 	BLI_array_fixedstack_free(kverts);
 }
@@ -2635,12 +2647,6 @@
 	for (lst = BLI_smallhash_iternew(ehash, &hiter, (uintptr_t *)&e); lst;
 	     lst = BLI_smallhash_iternext(&hiter, (uintptr_t *)&e))
 	{
-		if (kcd->only_select) {
-			if (!BM_elem_flag_test(e, BM_ELEM_SELECT)) {
-				continue;
-			}
-		}
-
 		sort_by_frac_along(lst, e);
 		for (ref = lst->first; ref; ref = ref->next) {
 			kfv = ref->ref;
@@ -2649,17 +2655,14 @@
 		}
 	}
 
+	if (kcd->select_result) {
+		EDBM_flag_disable_all(kcd->em, BM_ELEM_SELECT);
+	}
+
 	/* do cuts for each face */
 	for (lst = BLI_smallhash_iternew(fhash, &hiter, (uintptr_t *)&f); lst;
 	     lst = BLI_smallhash_iternext(&hiter, (uintptr_t *)&f))
 	{
-
-		if (kcd->only_select) {
-			if (!BM_elem_flag_test(f, BM_ELEM_SELECT)) {
-				continue;
-			}
-		}
-
 		knife_make_face_cuts(kcd, f, lst);
 	}
 
@@ -2836,6 +2839,9 @@
 	kcd->cut_through = !RNA_boolean_get(op->ptr, "use_occlude_geometry");
 	kcd->only_select = only_select;
 
+	/* can't usefully select resulting edges in face mode */
+	kcd->select_result = (kcd->em->selectmode != SCE_SELECT_FACE);
+
 	knife_pos_data_clear(&kcd->cur);
 	knife_pos_data_clear(&kcd->prev);