Mercurial > ldforge / file diff

--- a/src/actionsEdit.cc	Mon Jun 02 13:16:14 2014 +0300
+++ b/src/actionsEdit.cc	Mon Jun 02 13:35:41 2014 +0300
@@ -54,7 +54,7 @@
 
 	for (LDObjectPtr obj : objs)
 	{
-		if (data.length() > 0)
+		if (not data.isEmpty())
 			data += "\n";
 
 		data += obj->asText();
@@ -160,10 +160,9 @@
 //
 DEFINE_ACTION (SplitQuads, 0)
 {
-	LDObjectList objs = selection();
 	int num = 0;
 
-	for (LDObjectPtr obj : objs)
+	for (LDObjectPtr obj : selection())
 	{
 		if (obj->type() != OBJ_Quad)
 			continue;
@@ -239,10 +238,8 @@
 	{
 		for (LDObjectPtr obj : objs)
 		{
-			if (not obj->isColored())
-				continue;
-
-			obj->setColor (colnum);
+			if (obj->isColored())
+				obj->setColor (colnum);
 		}
 
 		refresh();
@@ -407,9 +404,7 @@
 //
 DEFINE_ACTION (Invert, CTRL_SHIFT (W))
 {
-	LDObjectList sel = selection();
-
-	for (LDObjectPtr obj : sel)
+	for (LDObjectPtr obj : selection())
 		obj->invert();
 
 	refresh();