r25044 by jghali - Code cleanup

scribus-commit scribus-commit at lists.scribus.net
Fri Mar 18 22:47:57 UTC 2022


Author: jghali
Date: Fri Mar 18 22:47:57 2022
New Revision: 25044

URL: http://scribus.net/websvn/listing.php?repname=Scribus&sc=1&rev=25044
Log:
Code cleanup

Modified:
    trunk/Scribus/scribus/desaxe/saxio.h
    trunk/Scribus/scribus/scribus.cpp

Modified: trunk/Scribus/scribus/desaxe/saxio.h
URL: http://scribus.net/websvn/diff.php?repname=Scribus&rev=25044&path=/trunk/Scribus/scribus/desaxe/saxio.h
==============================================================================
--- trunk/Scribus/scribus/desaxe/saxio.h	(original)
+++ trunk/Scribus/scribus/desaxe/saxio.h	Fri Mar 18 22:47:57 2022
@@ -33,7 +33,7 @@
 	    Writes an XML representation of this object using the default element tag, see below
 	 */
 	virtual void saxx(SaxHandler & ) const = 0;
-	virtual ~SaxIO() {}
+	virtual ~SaxIO() = default;
 	/**
 		Element tag which is usually used to serialize this object. "" means no elem is created by saxx()
 	 */

Modified: trunk/Scribus/scribus/scribus.cpp
URL: http://scribus.net/websvn/diff.php?repname=Scribus&rev=25044&path=/trunk/Scribus/scribus/scribus.cpp
==============================================================================
--- trunk/Scribus/scribus/scribus.cpp	(original)
+++ trunk/Scribus/scribus/scribus.cpp	Fri Mar 18 22:47:57 2022
@@ -2840,21 +2840,21 @@
 
 	scrRecentPasteActions.clear();
 	int max = qMin(m_prefsManager.appPrefs.scrapbookPrefs.numScrapbookCopies, scrapbookPalette->tempBView->objectMap.count());
-	if (max > 0)
-	{
-		auto it = scrapbookPalette->tempBView->objectMap.end();
-		QString strippedName;
-		for (int i = 0; i < max; ++i)
-		{
-			it--;
-			strippedName = it.key();
-			QPixmap pm = it.value().Preview;
-			scrRecentPasteActions.insert(strippedName, new ScrAction(ScrAction::RecentPaste, pm, QString(), QString("&%1 %2").arg(i + 1).arg(strippedName), QKeySequence(), this, strippedName));
-			connect( scrRecentPasteActions[strippedName], SIGNAL(triggeredData(QString)), this, SLOT(pasteRecent(QString)) );
-			scrMenuMgr->addMenuItemString(strippedName, "EditPasteRecent");
-		}
-		scrMenuMgr->addMenuItemStringsToRememberedMenu("EditPasteRecent", scrRecentPasteActions);
-	}
+	if (max <= 0)
+		return;
+
+	auto it = scrapbookPalette->tempBView->objectMap.end();
+	QString strippedName;
+	for (int i = 0; i < max; ++i)
+	{
+		it--;
+		strippedName = it.key();
+		QPixmap pm = it.value().Preview;
+		scrRecentPasteActions.insert(strippedName, new ScrAction(ScrAction::RecentPaste, pm, QString(), QString("&%1 %2").arg(i + 1).arg(strippedName), QKeySequence(), this, strippedName));
+		connect( scrRecentPasteActions[strippedName], SIGNAL(triggeredData(QString)), this, SLOT(pasteRecent(QString)) );
+		scrMenuMgr->addMenuItemString(strippedName, "EditPasteRecent");
+	}
+	scrMenuMgr->addMenuItemStringsToRememberedMenu("EditPasteRecent", scrRecentPasteActions);
 }
 
 void ScribusMainWindow::rebuildScrapbookMenu()
@@ -7651,11 +7651,9 @@
 			//can select to eg apply to even pages with a single odd page selected
 			if (pageSelection == 1 && (pageNum %2 != 0)) //Even, %2!=0 as 1st page is numbered 0
 				Apply_MasterPage(masterPageName, pageNum, false);
-			else
-			if (pageSelection == 2 && (pageNum %2 == 0)) //Odd, %2==0 as 1st page is numbered 0
+			else if (pageSelection == 2 && (pageNum %2 == 0)) //Odd, %2==0 as 1st page is numbered 0
 				Apply_MasterPage(masterPageName, pageNum, false);
-			else
-			if (pageSelection == 3) //All
+			else if (pageSelection == 3) //All
 				Apply_MasterPage(masterPageName, pageNum, false);
 		}
 	}
@@ -9662,7 +9660,7 @@
 				editMDialog = (MarkInsert*) new Mark2Mark(doc->marksList(), mrk, this);
 				QString l = mrk->getDestMarkName();
 				MarkType t = mrk->getDestMarkType();
-				Mark* m = doc->getMark(l,t);
+				Mark* m = doc->getMark(l, t);
 				editMDialog->setValues(mrk->label, m);
 			}
 			break;




More information about the scribus-commit mailing list