ldforge: comparison gui.cpp

-:aa40ce18f869
+:ada4679d5bce
 		return false;
 	// Clear the clipboard. However, its contents are dynamically allocated
 	// clones of LDObjects (cannot use pointers to real objects because the
 	// cut operation deletes them!), so we have to delete said objects first.
-	for (std::size_t i = 0; i < g_Clipboard.size(); ++i)
+	FOREACH (LDObject, *, obj, g_Clipboard)
-		delete g_Clipboard[i];
+		delete obj;
 	g_Clipboard.clear ();
 	for (std::size_t i = 0; i < objs.size(); ++i)
 		g_Clipboard.push_back (objs[i]->makeClone ());
 	return true;
-}
-// =============================================================================
-// * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
-// =============================================================================
-void ForgeWindow::deleteSelection () {
-	vector<LDObject*> objs = getSelectedObjects ();
-	// Delete the objects that were being selected
-	for (ulong i = 0; i < (ulong)objs.size(); ++i) {
-		LDObject* obj = objs[i];
-		g_CurrentFile->forgetObject (obj);
-		delete obj;
-	}
 }
 // =============================================================================
 // * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
 // =============================================================================
 // =============================================================================
 // * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
 // =============================================================================
 void ForgeWindow::slot_paste () {
-	for (std::size_t i = 0; i < g_Clipboard.size(); ++i) {
+	FOREACH (LDObject, *, obj, g_Clipboard)
-		LDObject* copy = g_Clipboard[i]->makeClone ();
+		g_CurrentFile->addObject (obj->makeClone ());
-		g_CurrentFile->addObject (copy);
-	}
 	refresh ();
 }
 // =============================================================================
 			continue;
 		LDSubfile* ref = static_cast<LDSubfile*> (obj);
 		// Get the inlined objects. These are clones of the subfile's contents.
-		vector<LDObject*> objs = ref->inlineContents (bDeep, ref->faMatrix,
+		vector<LDObject*> objs = ref->inlineContents (bDeep, ref->mMatrix,
 			ref->vPosition, true);
 		// Merge in the inlined objects
 		FOREACH (LDObject, *, inlineobj, objs)
 			g_CurrentFile->objects.insert (g_CurrentFile->objects.begin() + idx++, inlineobj);
 // * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
 // =============================================================================
 void ForgeWindow::slot_splitQuads () {
 	vector<LDObject*> objs = getSelectedObjects ();
-	// Delete the objects that were being selected
+	FOREACH (LDObject, *, obj, objs) {
-	for (ulong i = 0; i < (ulong)objs.size(); ++i) {
-		LDObject* obj = objs[i];
-		// Don't even consider non-quads
 		if (obj->getType() != OBJ_Quad)
 			continue;
 		static_cast<LDQuad*> (obj)->splitToTriangles ();
 	}
 	short dColor;
 	short dDefault = -1;
 	std::vector<LDObject*> objs = getSelectedObjects ();
-	// Try to get a consensus on the color used for use as a default.
+	// If all selected objects have the same color, said color is our default
-	for (ulong i = 0; i < objs.size(); ++i) {
+	// value to the color selection dialog.
-		LDObject* obj = objs[i];
+	FOREACH (LDObject, *, obj, objs) {
 		if (obj->dColor == -1)
-			continue; // doesn't use colors
+			continue; // doesn't use color
 		if (dDefault != -1 && obj->dColor != dDefault) {
-			// No unanimosity in object color, therefore we don't have a
+			// No consensus in object color, therefore we don't have a
 			// proper default value to use.
 			dDefault = -1;
 			break;
 		}
 		if (dDefault == -1)
 			dDefault = obj->dColor;
 	}
-	// Show the dialog to the user now and ask him for a color.
+	// Show the dialog to the user now and ask for a color.
 	if (ColorSelectDialog::staticDialog (dColor, dDefault, this)) {
-		for (ulong i = 0; i < objs.size(); ++i) {
+		FOREACH (LDObject, *, obj, objs)
-			LDObject* obj = objs[i];
 			if (obj->dColor != -1)
 				obj->dColor = dColor;
-		}
 		refresh ();
 	}
 }
 // =============================================================================
 void ForgeWindow::slot_makeBorders () {
 	vector<LDObject*> objs = getSelectedObjects ();
 	// Delete the objects that were being selected
-	for (std::size_t i = 0; i < objs.size(); ++i) {
+	FOREACH (LDObject, *, obj, objs) {
-		LDObject* obj = objs[i];
 		if (obj->getType() != OBJ_Quad && obj->getType() != OBJ_Triangle)
 			continue;
 		short dNumLines;
 		LDLine* lines[4];
 }
 // =============================================================================
 // * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
 // =============================================================================
+void ForgeWindow::deleteSelection () {
+	vector<LDObject*> objs = getSelectedObjects ();
+	// Delete the objects that were being selected
+	FOREACH (LDObject, *, obj, objs) {
+		g_CurrentFile->forgetObject (obj);
+		delete obj;
+	}
+}
+// =============================================================================
+// * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
+// =============================================================================
 void ForgeWindow::buildObjList () {
 	if (!g_CurrentFile)
 		return;
 	QList<QTreeWidgetItem*> qaItems;
 				zText.format ("%s %s, (",
 					ref->zFileName.chars(), ref->vPosition.getStringRep (true).chars());
 				for (short i = 0; i < 9; ++i)
 					zText.appendformat ("%s%s",
-						ftoa (ref->faMatrix[i]).chars(),
+						ftoa (ref->mMatrix[i]).chars(),
 						(i != 8) ? " " : "");
 				zText += ')';
 			}
 			break;

Mercurial > ldforge / file comparison

comparison: gui.cpp

gui.cpp