[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [58308] trunk/blender/source/blender/ editors/space_view3d/view3d_select.c: revert part of own commit r58304, gives different behavior when de-selecting with the lasso in some cases, will look into this after release.

[Campbell Barton]

Revision: 58308
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=58308
Author:   campbellbarton
Date:     2013-07-16 03:41:09 +0000 (Tue, 16 Jul 2013)
Log Message:
-----------
revert part of own commit r58304, gives different behavior when de-selecting with the lasso in some cases, will look into this after release.

Revision Links:
--------------
    http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=58304

Modified Paths:
--------------
    trunk/blender/source/blender/editors/space_view3d/view3d_select.c

Modified: trunk/blender/source/blender/editors/space_view3d/view3d_select.c
===================================================================
--- trunk/blender/source/blender/editors/space_view3d/view3d_select.c	2013-07-16 02:44:52 UTC (rev 58307)
+++ trunk/blender/source/blender/editors/space_view3d/view3d_select.c	2013-07-16 03:41:09 UTC (rev 58308)
@@ -625,8 +625,8 @@
 			    BLI_lasso_is_point_inside(data->mcords, data->moves, x0, y0, INT_MAX))
 			{
 				is_point_done = true;
-				if (data->select) ED_armature_ebone_selectflag_enable(ebone, BONE_ROOTSEL);
-				else              ED_armature_ebone_selectflag_disable(ebone, BONE_ROOTSEL);
+				if (data->select) ebone->flag |=  BONE_ROOTSEL;
+				else              ebone->flag &= ~BONE_ROOTSEL;
 			}
 		}
 
@@ -637,8 +637,8 @@
 			    BLI_lasso_is_point_inside(data->mcords, data->moves, x1, y1, INT_MAX))
 			{
 				is_point_done = true;
-				if (data->select) ED_armature_ebone_selectflag_enable(ebone, BONE_TIPSEL);
-				else              ED_armature_ebone_selectflag_disable(ebone, BONE_TIPSEL);
+				if (data->select) ebone->flag |=  BONE_TIPSEL;
+				else              ebone->flag &= ~BONE_TIPSEL;
 			}
 		}
 
@@ -646,7 +646,8 @@
 		if ((is_point_done == false) && (points_proj_tot == 2) &&
 		    BLI_lasso_is_edge_inside(data->mcords, data->moves, x0, y0, x1, y1, INT_MAX))
 		{
-			ED_armature_ebone_select_set(ebone, data->select);
+			if (data->select) ebone->flag |=  (BONE_SELECTED | BONE_TIPSEL | BONE_ROOTSEL);
+			else              ebone->flag &= ~(BONE_SELECTED | BONE_TIPSEL | BONE_ROOTSEL);
 			data->is_change = true;
 		}
 
@@ -1909,14 +1910,14 @@
 			if ((select == false) || ((ebone->flag & BONE_UNSELECTABLE) == 0)) {
 				if (index & BONESEL_TIP) {
 					ebone->flag |= BONE_DONE;
-					if (select) ED_armature_ebone_selectflag_enable(ebone, BONE_TIPSEL);
-					else        ED_armature_ebone_selectflag_disable(ebone, BONE_TIPSEL);
+					if (select) ebone->flag |= BONE_TIPSEL;
+					else ebone->flag &= ~BONE_TIPSEL;
 				}
 				
 				if (index & BONESEL_ROOT) {
 					ebone->flag |= BONE_DONE;
-					if (select) ED_armature_ebone_selectflag_enable(ebone, BONE_ROOTSEL);
-					else        ED_armature_ebone_selectflag_disable(ebone, BONE_ROOTSEL);
+					if (select) ebone->flag |= BONE_ROOTSEL;
+					else ebone->flag &= ~BONE_ROOTSEL;
 				}
 			}
 		}
@@ -1938,7 +1939,10 @@
 			if (index & BONESEL_BONE) {
 				if ((select == false) || ((ebone->flag & BONE_UNSELECTABLE) == 0)) {
 					if (!(ebone->flag & BONE_DONE)) {
-						ED_armature_ebone_select_set(ebone, select);
+						if (select)
+							ebone->flag |= (BONE_ROOTSEL | BONE_TIPSEL | BONE_SELECTED);
+						else
+							ebone->flag &= ~(BONE_ROOTSEL | BONE_TIPSEL | BONE_SELECTED);
 					}
 				}
 			}
@@ -2598,12 +2602,16 @@
 
 	if (len_squared_v2v2(data->mval_fl, screen_co) <= data->radius_squared) {
 		if (head) {
-			if (data->select) ED_armature_ebone_selectflag_enable(ebone, BONE_ROOTSEL);
-			else              ED_armature_ebone_selectflag_disable(ebone, BONE_ROOTSEL);
+			if (data->select)
+				ebone->flag |= BONE_ROOTSEL;
+			else 
+				ebone->flag &= ~BONE_ROOTSEL;
 		}
 		else {
-			if (data->select) ED_armature_ebone_selectflag_enable(ebone, BONE_TIPSEL);
-			else              ED_armature_ebone_selectflag_disable(ebone, BONE_TIPSEL);
+			if (data->select)
+				ebone->flag |= BONE_TIPSEL;
+			else 
+				ebone->flag &= ~BONE_TIPSEL;
 		}
 		return 1;
 	}
@@ -2644,7 +2652,8 @@
 		if ((is_point_done == false) && (points_proj_tot == 2) &&
 		    edge_inside_circle(data->mval_fl, data->radius, screen_co_a, screen_co_b))
 		{
-			ED_armature_ebone_select_set(ebone, data->select);
+			if (data->select) ebone->flag |=  (BONE_SELECTED | BONE_TIPSEL | BONE_ROOTSEL);
+			else              ebone->flag &= ~(BONE_SELECTED | BONE_TIPSEL | BONE_ROOTSEL);
 			data->is_change = true;
 		}