Author: craig
Date: Thu Feb  3 20:45:13 2022
New Revision: 24887

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

Modified:
    trunk/Scribus/scribus/fonts/sfnt.cpp
    trunk/Scribus/scribus/pageitempointer.cpp
    trunk/Scribus/scribus/scribus.cpp
    trunk/Scribus/scribus/scribusapp.cpp
    trunk/Scribus/scribus/scribusdoc.cpp
    trunk/Scribus/scribus/ui/scrspinbox.cpp

Modified: trunk/Scribus/scribus/fonts/sfnt.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24887&path=/trunk/Scribus/scribus/fonts/sfnt.cpp
==============================================================================
--- trunk/Scribus/scribus/fonts/sfnt.cpp        (original)
+++ trunk/Scribus/scribus/fonts/sfnt.cpp        Thu Feb  3 20:45:13 2022
@@ -725,7 +725,7 @@
                                symbolFormat = format;
                                continue;
                        }
-                       format = 1; // no such format
+//                     format = 1; // no such format
                }
                // If no unicode cmap was found, fallback on ms symbol cmap
                uint format = unicodeFormat;

Modified: trunk/Scribus/scribus/pageitempointer.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24887&path=/trunk/Scribus/scribus/pageitempointer.cpp
==============================================================================
--- trunk/Scribus/scribus/pageitempointer.cpp   (original)
+++ trunk/Scribus/scribus/pageitempointer.cpp   Thu Feb  3 20:45:13 2022
@@ -17,7 +17,7 @@
 void PageItemPointerDB::addToDB(const QString & key, PageItem * value)
 {
        insert(key,value);
-       connect ( value,SIGNAL( destroyed ( QObject* ) ),this,SLOT( 
removeFromDB ( QObject* ) ) );
+       connect(value, SIGNAL(destroyed(QObject*)), this, 
SLOT(removeFromDB(QObject*)));
 }
 
 void PageItemPointerDB::removeFromDB ( QObject * obj )

Modified: trunk/Scribus/scribus/scribus.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24887&path=/trunk/Scribus/scribus/scribus.cpp
==============================================================================
--- trunk/Scribus/scribus/scribus.cpp   (original)
+++ trunk/Scribus/scribus/scribus.cpp   Thu Feb  3 20:45:13 2022
@@ -788,8 +788,8 @@
        connect(docCheckerPalette, SIGNAL(selectElement(PageItem*,bool,int)), 
this, SLOT(selectItemFromOutlines(PageItem*,bool,int)));
        connect(docCheckerPalette, SIGNAL(selectPage(int)), this, 
SLOT(selectPagesFromOutlines(int)));
        connect(docCheckerPalette, SIGNAL(selectMasterPage(QString)), this, 
SLOT(editMasterPagesStart(QString)));
-       connect(outlinePalette, SIGNAL(selectElementByItem(PageItem *, bool)), 
this, SLOT(selectItemsFromOutlines(PageItem*,bool)));
-       connect(outlinePalette, SIGNAL(editElementByItem(PageItem *)), this, 
SLOT(editItemsFromOutlines(PageItem *)));
+       connect(outlinePalette, SIGNAL(selectElementByItem(PageItem*,bool)), 
this, SLOT(selectItemsFromOutlines(PageItem*,bool)));
+       connect(outlinePalette, SIGNAL(editElementByItem(PageItem*)), this, 
SLOT(editItemsFromOutlines(PageItem*)));
        connect(outlinePalette, SIGNAL(selectPage(int)), this, 
SLOT(selectPagesFromOutlines(int)));
        connect(outlinePalette, SIGNAL(selectMasterPage(QString)), this, 
SLOT(editMasterPagesStart(QString)));
        connect(nodePalette, SIGNAL(paletteClosed()), this, SLOT(slotSelect()));
@@ -798,7 +798,7 @@
 
        connect(bookmarkPalette->BView, SIGNAL(markMoved()), this, 
SLOT(StoreBookmarks()));
        connect(bookmarkPalette->BView, SIGNAL(changed()), this, 
SLOT(slotDocCh()));
-       connect(bookmarkPalette->BView, SIGNAL(selectElement(PageItem *, 
bool)), this, SLOT(selectItemsFromOutlines(PageItem *, bool)));
+       connect(bookmarkPalette->BView, SIGNAL(selectElement(PageItem*,bool)), 
this, SLOT(selectItemsFromOutlines(PageItem*,bool)));
        // guides
        connect(scrActions["pageManageGuides"], SIGNAL(toggled(bool)), 
guidePalette, SLOT(setPaletteShown(bool)));
        connect(guidePalette, SIGNAL(paletteShown(bool)), 
scrActions["pageManageGuides"], SLOT(setChecked(bool)));
@@ -831,8 +831,8 @@
        
scrapbookPalette->setOpenScrapbooks(m_prefsManager.appPrefs.scrapbookPrefs.RecentScrapbooks);
        rebuildRecentPasteMenu();
        connect(scrapbookPalette, SIGNAL(updateRecentMenue()), this, 
SLOT(rebuildRecentPasteMenu()));
-       connect(ScCore->fileWatcher, SIGNAL(dirChanged(QString )), 
scrapbookPalette, SLOT(reloadLib(QString )));
-       connect(ScCore->fileWatcher, SIGNAL(dirDeleted(QString )), 
scrapbookPalette, SLOT(closeOnDel(QString )));
+       connect(ScCore->fileWatcher, SIGNAL(dirChanged(QString)), 
scrapbookPalette, SLOT(reloadLib(QString)));
+       connect(ScCore->fileWatcher, SIGNAL(dirDeleted(QString)), 
scrapbookPalette, SLOT(closeOnDel(QString)));
 }
 
 bool ScribusMainWindow::warningVersion(QWidget *parent)
@@ -1440,7 +1440,7 @@
 
        statusBar()->addPermanentWidget(unitSwitcher,0);
        statusBar()->addPermanentWidget(mainWindowProgressBar, 0);
-       connect(statusBar(), SIGNAL(messageChanged(const QString &)), this, 
SLOT(setTempStatusBarText(const QString &)));
+       connect(statusBar(), SIGNAL(messageChanged(QString)), this, 
SLOT(setTempStatusBarText(QString)));
 
 }
 
@@ -1819,7 +1819,7 @@
                int windowCount = windows.count();
                for ( int i = 0; i < windowCount; ++i )
                {
-                       tw = qobject_cast<ScribusWin *>(windows.at(i));
+//                     tw = qobject_cast<ScribusWin *>(windows.at(i));
                        QMdiSubWindow *tws = windows.at(i);
                        ScribusWin* scw = dynamic_cast<ScribusWin 
*>(tws->widget());
                        if (scw)
@@ -2625,7 +2625,7 @@
        connect(view, SIGNAL(HaveSel()), this, SLOT(HaveNewSel()), 
Qt::UniqueConnection);
        connect(view, SIGNAL(DocChanged()), this, SLOT(slotDocCh()), 
Qt::UniqueConnection);
        connect(view, SIGNAL(MousePos(double,double)), this, 
SLOT(setStatusBarMousePosition(double,double)), Qt::UniqueConnection);
-       connect(view, SIGNAL(ItemCharStyle(const CharStyle&)), contentPalette, 
SLOT(update(const CharStyle&)), Qt::UniqueConnection);
+       connect(view, SIGNAL(ItemCharStyle(CharStyle)), contentPalette, 
SLOT(update(CharStyle)), Qt::UniqueConnection);
        connect(view, SIGNAL(ItemTextEffects(int)), this, 
SLOT(setStyleEffects(int)), Qt::UniqueConnection);
        connect(view, SIGNAL(ItemTextAlign(int)), this, 
SLOT(setAlignmentValue(int)), Qt::UniqueConnection);
        connect(view, 
SIGNAL(LoadElem(QString,double,double,bool,bool,ScribusDoc*,ScribusView*)), 
this, 
SLOT(slotElemRead(QString,double,double,bool,bool,ScribusDoc*,ScribusView*)), 
Qt::UniqueConnection);
@@ -7004,7 +7004,7 @@
        }
        else
        {
-               QDir di = QDir();
+//             QDir di = QDir();
                if (!doc->m_Selection->isEmpty())
                        filename = QDir::currentPath() + "/" + 
doc->documentFileName() + "_selection.eps";
                else
@@ -7175,7 +7175,7 @@
                        std::vector<int> pageNs2;
                        pageNs2.clear();
                        pageNs2.push_back(pageNs[aa]);
-                       pageNumbersSize = pageNs2.size();
+//                     pageNumbersSize = pageNs2.size();
                        QImage thumb(10, 10, 
QImage::Format_ARGB32_Premultiplied);
                        if (doc->pdfOptions().Thumbnails)
                                thumb = allThumbs[pageNs[aa]];

Modified: trunk/Scribus/scribus/scribusapp.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24887&path=/trunk/Scribus/scribus/scribusapp.cpp
==============================================================================
--- trunk/Scribus/scribus/scribusapp.cpp        (original)
+++ trunk/Scribus/scribus/scribusapp.cpp        Thu Feb  3 20:45:13 2022
@@ -455,15 +455,15 @@
 #endif
 
        langs.append(QLocale::system().name());
-
-       // remove duplicate entries...
-       QStringList::Iterator it = langs.end();
-       while (it != langs.begin())
-       {
-               --it;
-               if (langs.count(*it) > 1)
-                       it = langs.erase(it);
-       }
+       langs.removeDuplicates();
+//     // remove duplicate entries...
+//     QStringList::Iterator it = langs.end();
+//     while (it != langs.begin())
+//     {
+//             --it;
+//             if (langs.count(*it) > 1)
+//                     it = langs.erase(it);
+//     }
        return langs;
 }
 

Modified: trunk/Scribus/scribus/scribusdoc.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24887&path=/trunk/Scribus/scribus/scribusdoc.cpp
==============================================================================
--- trunk/Scribus/scribus/scribusdoc.cpp        (original)
+++ trunk/Scribus/scribus/scribusdoc.cpp        Thu Feb  3 20:45:13 2022
@@ -6013,11 +6013,12 @@
        int counter = 
pageSets()[m_docPrefsData.docSetupPrefs.pagePositioning].FirstPage;
        int rowcounter = 0;
        double maxYPos=0.0, maxXPos=0.0;
-       double currentXPos=m_docPrefsData.displayPrefs.scratch.left(), 
currentYPos=m_docPrefsData.displayPrefs.scratch.top(), 
lastYPos=Pages->at(0)->initialHeight();
+       double currentXPos=m_docPrefsData.displayPrefs.scratch.left();
+       double currentYPos=m_docPrefsData.displayPrefs.scratch.top();
+       double lastYPos=Pages->at(0)->initialHeight();
 //     currentXPos += (pageWidth+pageSets[currentPageLayout].GapHorizontal) * 
counter;
        currentXPos += 
(m_docPrefsData.docSetupPrefs.pageWidth+m_docPrefsData.displayPrefs.pageGapHorizontal)
 * counter;
 
-       lastYPos = Pages->at(0)->initialHeight();
        ScPage* page;
        int docPageCount=Pages->count();
        for (int i = 0; i < docPageCount; ++i)

Modified: trunk/Scribus/scribus/ui/scrspinbox.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24887&path=/trunk/Scribus/scribus/ui/scrspinbox.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/scrspinbox.cpp     (original)
+++ trunk/Scribus/scribus/ui/scrspinbox.cpp     Thu Feb  3 20:45:13 2022
@@ -45,8 +45,9 @@
        setLocale(LocaleManager::instance().userPreferredLocale());
        setSingleStep(1.0);
        lineEdit()->setValidator(nullptr);
-       disconnect(this, SIGNAL(textChanged(const QString &)), this, 
SLOT(textChanged()));
-       connect(this, SIGNAL(textChanged(const QString &)), this, 
SLOT(textChanged()));
+// just for testing
+//     disconnect(this, SIGNAL(textChanged(const QString &)), this, 
SLOT(textChanged()));
+//     connect(this, SIGNAL(textChanged(const QString &)), this, 
SLOT(textChanged()));
        installEventFilter(this);
 }
 


_______________________________________________
scribus-commit mailing list
[email protected]
http://lists.scribus.net/mailman/listinfo/scribus-commit

Reply via email to