Author: jghali
Date: Sat Oct 24 16:23:13 2020
New Revision: 24096

URL: http://scribus.net/websvn/listing.php?repname=Scribus&sc=1&rev=24096
Log:
Fix more compiler warnings

Modified:
    branches/Version14x/Scribus/scribus/bookmwin.cpp
    branches/Version14x/Scribus/scribus/exif.cpp
    branches/Version14x/Scribus/scribus/fileunzip.cpp
    branches/Version14x/Scribus/scribus/pageitem.cpp
    branches/Version14x/Scribus/scribus/pdflib_core.cpp
    branches/Version14x/Scribus/scribus/printdialog.cpp
    branches/Version14x/Scribus/scribus/pslib.cpp
    branches/Version14x/Scribus/scribus/scfonts.cpp
    branches/Version14x/Scribus/scribus/scimage.cpp
    branches/Version14x/Scribus/scribus/scimgdataloader.cpp
    branches/Version14x/Scribus/scribus/scimgdataloader_psd.cpp
    branches/Version14x/Scribus/scribus/scimgdataloader_tiff.cpp
    branches/Version14x/Scribus/scribus/scribus.cpp
    branches/Version14x/Scribus/scribus/scribusdoc.cpp
    branches/Version14x/Scribus/scribus/smlinestyle.cpp

Modified: branches/Version14x/Scribus/scribus/bookmwin.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24096&path=/branches/Version14x/Scribus/scribus/bookmwin.cpp
==============================================================================
--- branches/Version14x/Scribus/scribus/bookmwin.cpp    (original)
+++ branches/Version14x/Scribus/scribus/bookmwin.cpp    Sat Oct 24 16:23:13 2020
@@ -150,7 +150,6 @@
        text.clear();
        title.clear();
 
-       const StoryText& itemText = item->itemText;
        for (int i = 0; i < item->itemText.length(); ++i)
        {
                ch = item->itemText.text(i);

Modified: branches/Version14x/Scribus/scribus/exif.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24096&path=/branches/Version14x/Scribus/scribus/exif.cpp
==============================================================================
--- branches/Version14x/Scribus/scribus/exif.cpp        (original)
+++ branches/Version14x/Scribus/scribus/exif.cpp        Sat Oct 24 16:23:13 2020
@@ -777,12 +777,12 @@
 //--------------------------------------------------------------------------
 void ExifData::process_SOFn ( const uchar * Data, int marker )
 {
-       int data_precision, num_components;
-
-       data_precision = Data[2];
+//     int data_precision, num_components;
+
+//     data_precision = Data[2];
        ExifData::Height = Get16m ( Data+3 );
        ExifData::Width = Get16m ( Data+5 );
-       num_components = Data[7];
+       int num_components = Data[7];
 
        if ( num_components == 3 )
                ExifData::IsColor = 1;

Modified: branches/Version14x/Scribus/scribus/fileunzip.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24096&path=/branches/Version14x/Scribus/scribus/fileunzip.cpp
==============================================================================
--- branches/Version14x/Scribus/scribus/fileunzip.cpp   (original)
+++ branches/Version14x/Scribus/scribus/fileunzip.cpp   Sat Oct 24 16:23:13 2020
@@ -174,14 +174,17 @@
 {
        QChar p;
        QString buffer;
+
        int len = newdir.length();
-       
        if (len <= 0)
                return 0;
        buffer = newdir;
 
-       if (buffer.endsWith("/") || buffer.endsWith("\\"))
-               buffer.left(len - 1);
+       while (buffer.endsWith("/") || buffer.endsWith("\\"))
+               buffer = buffer.left(len - 1);
+
+       if (buffer.length() <= 0)
+               return 0;
 
        if (mymkdir(buffer) == 0)
                return 1;

Modified: branches/Version14x/Scribus/scribus/pageitem.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24096&path=/branches/Version14x/Scribus/scribus/pageitem.cpp
==============================================================================
--- branches/Version14x/Scribus/scribus/pageitem.cpp    (original)
+++ branches/Version14x/Scribus/scribus/pageitem.cpp    Sat Oct 24 16:23:13 2020
@@ -3561,9 +3561,11 @@
 
 void PageItem::restorePStyle(SimpleState *state, bool isUndo)
 {
-       int styleid = state->getInt("OLD_STYLE");
-       if (!isUndo)
-               styleid = state->getInt("NEW_STYLE");
+       Q_UNUSED(state);
+       Q_UNUSED(isUndo);
+       //int styleid = state->getInt("OLD_STYLE");
+       //if (!isUndo)
+       //      styleid = state->getInt("NEW_STYLE");
        //will be done later with other text-undo:
        //      m_Doc->chAbStyle(this, styleid);
 }

Modified: branches/Version14x/Scribus/scribus/pdflib_core.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24096&path=/branches/Version14x/Scribus/scribus/pdflib_core.cpp
==============================================================================
--- branches/Version14x/Scribus/scribus/pdflib_core.cpp (original)
+++ branches/Version14x/Scribus/scribus/pdflib_core.cpp Sat Oct 24 16:23:13 2020
@@ -7620,7 +7620,6 @@
        QString tmp;
        int ResO;
        BookMItem* ip;
-       QTreeWidgetItem* pp;
        QString Inhal = "";
        QMap<int,QString> Inha;
        if ((Bvie->topLevelItemCount() != 0) && (Options.Bookmarks) && 
(BookMinUse))
@@ -7628,7 +7627,6 @@
                int Basis = ObjCounter - 1;
                Outlines.Count = Bvie->topLevelItemCount();
                ip = (BookMItem*)Bvie->topLevelItem(0);
-               pp = Bvie->topLevelItem(0);
                Outlines.First = ip->ItemNr+Basis;
                Outlines.Last  = ((BookMItem*) 
Bvie->topLevelItem(Outlines.Count - 1))->ItemNr+Basis;
                QTreeWidgetItemIterator it(Bvie);

Modified: branches/Version14x/Scribus/scribus/printdialog.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24096&path=/branches/Version14x/Scribus/scribus/printdialog.cpp
==============================================================================
--- branches/Version14x/Scribus/scribus/printdialog.cpp (original)
+++ branches/Version14x/Scribus/scribus/printdialog.cpp Sat Oct 24 16:23:13 2020
@@ -539,7 +539,6 @@
        GcR->setChecked(m_doc->Print_Options.doGCR);
        ClipMarg->setChecked(m_doc->Print_Options.doClip);
        spotColors->setChecked(!m_doc->Print_Options.useSpotColors);
-       bool psPrinter = 
PrinterUtil::isPostscriptPrinter(PrintDest->currentText()) || outputToFile();
        docBleeds->setChecked(m_doc->Print_Options.useDocBleeds);
        if (docBleeds->isChecked())
        {

Modified: branches/Version14x/Scribus/scribus/pslib.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24096&path=/branches/Version14x/Scribus/scribus/pslib.cpp
==============================================================================
--- branches/Version14x/Scribus/scribus/pslib.cpp       (original)
+++ branches/Version14x/Scribus/scribus/pslib.cpp       Sat Oct 24 16:23:13 2020
@@ -327,7 +327,6 @@
 bool PSLib::PutInterleavedImageMaskToStream(const QByteArray& image, const 
QByteArray& mask, bool gray)
 {
        int pending = 0;
-       int bIndex  = 0;
        unsigned char bytes[1505];
        const unsigned char* imageData = (const unsigned char*) 
image.constData();
        const unsigned char* maskData  = (const unsigned char*) 
mask.constData();
@@ -345,7 +344,6 @@
 
        for (int i = 0; i < pixels; ++i)
        {
-               bIndex = 4 *i;
                bytes[pending++] = maskData [i];
                bytes[pending++] = *imageData++; // cyan/black
                if (channels > 1)

Modified: branches/Version14x/Scribus/scribus/scfonts.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24096&path=/branches/Version14x/Scribus/scribus/scfonts.cpp
==============================================================================
--- branches/Version14x/Scribus/scribus/scfonts.cpp     (original)
+++ branches/Version14x/Scribus/scribus/scfonts.cpp     Sat Oct 24 16:23:13 2020
@@ -117,8 +117,9 @@
                return;
        FT_Library library = NULL;
        QString pathfile, fullpath;
-       bool error;
-       error = FT_Init_FreeType( &library );
+//     bool error;
+//     error =
+       FT_Init_FreeType( &library );
        QString pathname(path);
        if ( !pathname.endsWith("/") )
                pathname += "/";
@@ -177,12 +178,12 @@
                                ext = ext2;
                        if ((ext == "ttc") || (ext == "dfont") || (ext == 
"pfa") || (ext == "pfb") || (ext == "ttf") || (ext == "otf"))
                        {
-                               error = AddScalableFont(pathfile, library, 
DocName);
+                               AddScalableFont(pathfile, library, DocName);
                        }
 #ifdef Q_OS_MAC
                        else if (ext.isEmpty() && DocName.isEmpty())
                        {
-                               error = AddScalableFont(pathfile, library, 
DocName);
+                               bool error = AddScalableFont(pathfile, library, 
DocName);
                                if (error)
                                        error = AddScalableFont(pathfile + 
"/..namedfork/rsrc",library, DocName);
                        }

Modified: branches/Version14x/Scribus/scribus/scimage.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24096&path=/branches/Version14x/Scribus/scribus/scimage.cpp
==============================================================================
--- branches/Version14x/Scribus/scribus/scimage.cpp     (original)
+++ branches/Version14x/Scribus/scribus/scimage.cpp     Sat Oct 24 16:23:13 2020
@@ -2149,8 +2149,7 @@
        // gsRes - is the resolution that ghostscript will render at
        bool isCMYK = false;
        bool ret = false;
-       bool inputProfisEmbedded = false;
-       bool stdProof = false;
+//     bool inputProfisEmbedded = false;
        if (realCMYK != 0)
                *realCMYK = false;
        bool bilevel = false;
@@ -2245,7 +2244,7 @@
                if ((embeddedProfile.size() > 0 ) && (useEmbedded))
                {
                        inputProf = 
ScColorMgmtEngine::openProfileFromMem(embeddedProfile);
-                       inputProfisEmbedded = true;
+               //      inputProfisEmbedded = true;
                }
                else
                {
@@ -2257,7 +2256,7 @@
                                if 
(ScCore->InputProfilesCMYK.contains(cmSettings.profileName()) && 
(cmSettings.profileName() != 
cmSettings.doc()->CMSSettings.DefaultImageCMYKProfile))
                                {
                                        imgInfo.profileName = 
cmSettings.profileName();
-                                       inputProfisEmbedded = true;
+                               //      inputProfisEmbedded = true;
                                        profilePath = 
ScCore->InputProfilesCMYK[imgInfo.profileName];
                                        inputProf = 
ScColorMgmtEngine::openProfileFromFile(profilePath);
                                }
@@ -2265,7 +2264,7 @@
                                {
                                        inputProf = 
cmSettings.doc()->DocInputImageCMYKProf;
                                        imgInfo.profileName = 
cmSettings.doc()->CMSSettings.DefaultImageCMYKProfile;
-                                       inputProfisEmbedded = false;
+                               //      inputProfisEmbedded = false;
                                }
                        }
                        else if (bilevel && (reqType == CMYKData))
@@ -2274,21 +2273,21 @@
                        {
                                imgInfo.profileName = cmSettings.profileName();
                                profilePath = 
ScCore->InputProfiles[imgInfo.profileName].toLocal8Bit();
-                               inputProfisEmbedded = true;
+                               //inputProfisEmbedded = true;
                                inputProf = 
ScColorMgmtEngine::openProfileFromFile(profilePath);
                        }
                        else
                        {
                                inputProf = 
cmSettings.doc()->DocInputImageRGBProf;
                                imgInfo.profileName = 
cmSettings.doc()->CMSSettings.DefaultImageRGBProfile;
-                               inputProfisEmbedded = false;
+                       //      inputProfisEmbedded = false;
                        }
                }
        }
        else if ((useProf && embeddedProfile.size() > 0) && (useEmbedded))
        {
                inputProf = 
ScColorMgmtEngine::openProfileFromMem(embeddedProfile);
-               inputProfisEmbedded = true;
+       //      inputProfisEmbedded = true;
        }
        else if (useProf && isCMYK)
                inputProf = ScCore->defaultCMYKProfile;
@@ -2370,7 +2369,6 @@
                                                else
                                                        xform = 
cmSettings.rgbImageProofingTransform();
                                                
xform.changeBufferFormat(inputProfFormat, Format_BGRA_8);
-                                               stdProof = true;
                                        }
                                        else
                                                xform = 
ScColorMgmtEngine::createProofingTransform(inputProf, inputProfFormat,

Modified: branches/Version14x/Scribus/scribus/scimgdataloader.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24096&path=/branches/Version14x/Scribus/scribus/scimgdataloader.cpp
==============================================================================
--- branches/Version14x/Scribus/scribus/scimgdataloader.cpp     (original)
+++ branches/Version14x/Scribus/scribus/scimgdataloader.cpp     Sat Oct 24 
16:23:13 2020
@@ -246,7 +246,7 @@
                                        s.readRawData(buffer, thsize);
                                        QImage imth;
                                        imth.loadFromData((const uchar*)buffer, 
thsize, "JPEG");
-                                       
imth.convertToFormat(QImage::Format_ARGB32);
+                                       imth = 
imth.convertToFormat(QImage::Format_ARGB32);
                                        if (resID == 0x0409)
                                                
m_imageInfoRecord.exifInfo.thumbnail = imth.rgbSwapped();
                                        else

Modified: branches/Version14x/Scribus/scribus/scimgdataloader_psd.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24096&path=/branches/Version14x/Scribus/scribus/scimgdataloader_psd.cpp
==============================================================================
--- branches/Version14x/Scribus/scribus/scimgdataloader_psd.cpp (original)
+++ branches/Version14x/Scribus/scribus/scimgdataloader_psd.cpp Sat Oct 24 
16:23:13 2020
@@ -132,7 +132,6 @@
 bool ScImgDataLoader_PSD::loadPicture(const QString& fn, int /*page*/, int 
res, bool thumbnail)
 {
        bool isCMYK = false;
-       float xres = 72.0, yres = 72.0;
        if (!QFile::exists(fn))
                return false;
        bool valid = m_imageInfoRecord.isRequest;
@@ -181,8 +180,6 @@
                        m_imageInfoRecord.colorspace = ColorSpaceGray;
                else if (header.color_mode == CM_DUOTONE)
                        m_imageInfoRecord.colorspace = ColorSpaceDuotone;
-               xres = m_imageInfoRecord.xres;
-               yres = m_imageInfoRecord.yres;
                f.close();
                if (thumbnail)
                {
@@ -285,7 +282,7 @@
        uint tmp;
        uint cdataStart;
        uint ressourceDataLen;
-       uint startRessource;
+//     uint startRessource;
 
        s.device()->seek( dataOffset );
 
@@ -481,7 +478,7 @@
        }
        s.device()->seek( cdataStart + tmp );
        s >> ressourceDataLen;
-       startRessource = s.device()->pos();
+//     startRessource = s.device()->pos();
        if (ressourceDataLen != 0)
                parseRessourceData(s, header, ressourceDataLen);
        return true;
@@ -985,8 +982,8 @@
                        startSrcX = 0;
                        startDstX = layerInfo[layer].xpos;
                }
-               unsigned int startSrcYm, startSrcXm, startDstYm, startDstXm;
-               if (layerInfo[layer].maskYpos < 0)
+               unsigned int startSrcXm; //, startSrcYm, startDstYm, startDstXm;
+       /*      if (layerInfo[layer].maskYpos < 0)
                {
                        startSrcYm = abs(layerInfo[layer].maskYpos);
                        startDstYm = 0;
@@ -995,16 +992,16 @@
                {
                        startSrcYm = 0;
                        startDstYm = layerInfo[layer].maskYpos;
-               }
+               }*/
                if (layerInfo[layer].maskXpos < 0)
                {
                        startSrcXm = abs(layerInfo[layer].maskXpos);
-                       startDstXm = 0;
+       //              startDstXm = 0;
                }
                else
                {
                        startSrcXm = 0;
-                       startDstXm = layerInfo[layer].maskXpos;
+       //              startDstXm = layerInfo[layer].maskXpos;
                }
                QString layBlend2 = layerInfo[layer].blend;
                if ((m_imageInfoRecord.isRequest) && 
(m_imageInfoRecord.RequestProps.contains(layer)))

Modified: branches/Version14x/Scribus/scribus/scimgdataloader_tiff.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24096&path=/branches/Version14x/Scribus/scribus/scimgdataloader_tiff.cpp
==============================================================================
--- branches/Version14x/Scribus/scribus/scimgdataloader_tiff.cpp        
(original)
+++ branches/Version14x/Scribus/scribus/scimgdataloader_tiff.cpp        Sat Oct 
24 16:23:13 2020
@@ -1439,8 +1439,8 @@
                        startSrcX = 0;
                        startDstX = layerInfo[layer].xpos;
                }
-               unsigned int startSrcYm, startSrcXm, startDstYm, startDstXm;
-               if (layerInfo[layer].maskYpos < 0)
+               unsigned int startSrcXm; //, startSrcYm, startDstYm, startDstXm;
+       /*      if (layerInfo[layer].maskYpos < 0)
                {
                        startSrcYm = abs(layerInfo[layer].maskYpos);
                        startDstYm = 0;
@@ -1449,16 +1449,16 @@
                {
                        startSrcYm = 0;
                        startDstYm = layerInfo[layer].maskYpos;
-               }
+               }*/
                if (layerInfo[layer].maskXpos < 0)
                {
                        startSrcXm = abs(layerInfo[layer].maskXpos);
-                       startDstXm = 0;
+               //      startDstXm = 0;
                }
                else
                {
                        startSrcXm = 0;
-                       startDstXm = layerInfo[layer].maskXpos;
+               //      startDstXm = layerInfo[layer].maskXpos;
                }
                QString layBlend2 = layerInfo[layer].blend;
                if ((m_imageInfoRecord.isRequest) && 
(m_imageInfoRecord.RequestProps.contains(layer)))

Modified: branches/Version14x/Scribus/scribus/scribus.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24096&path=/branches/Version14x/Scribus/scribus/scribus.cpp
==============================================================================
--- branches/Version14x/Scribus/scribus/scribus.cpp     (original)
+++ branches/Version14x/Scribus/scribus/scribus.cpp     Sat Oct 24 16:23:13 2020
@@ -8637,7 +8637,7 @@
                                QMessageBox msgBox;
                                QPushButton *abortButton = 
msgBox.addButton(QMessageBox::Cancel);
                                QPushButton *lockButton = 
msgBox.addButton(tr("&Lock All"), QMessageBox::AcceptRole);
-                               QPushButton *unlockButton = 
msgBox.addButton(tr("&Unlock All"), QMessageBox::AcceptRole);
+                               msgBox.addButton(tr("&Unlock All"), 
QMessageBox::AcceptRole);
                                msgBox.setIcon(QMessageBox::Warning);
                                msgBox.setWindowTitle(CommonStrings::trWarning);
                                msgBox.setText( tr("Some objects are locked."));
@@ -8647,7 +8647,6 @@
                                else if (msgBox.clickedButton() == lockButton)
                                        lockObject = true;
                                modifyLock = true;
-                               unlockButton = NULL;    // just to silence the 
compiler
                        }
                }
                doc->itemSelection_GroupObjects(modifyLock, lockObject);

Modified: branches/Version14x/Scribus/scribus/scribusdoc.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24096&path=/branches/Version14x/Scribus/scribus/scribusdoc.cpp
==============================================================================
--- branches/Version14x/Scribus/scribus/scribusdoc.cpp  (original)
+++ branches/Version14x/Scribus/scribus/scribusdoc.cpp  Sat Oct 24 16:23:13 2020
@@ -5135,11 +5135,9 @@
                return false;
        QMap<uint, DocumentSection>::Iterator itprev=sections.begin();
        QMap<uint, DocumentSection>::Iterator it=itprev;
-       uint currMinIndex = itprev.value().fromindex;
        uint currMaxIndex = itprev.value().toindex;
        for ( ; it != sections.end(); ++it )
        {
-               currMinIndex=it.value().fromindex;
                currMaxIndex=it.value().toindex;
 
                if (it.key()!=number)
@@ -5162,14 +5160,12 @@
 
 int ScribusDoc::getSectionKeyForPageIndex(const uint pageIndex) const
 {
-       bool found=false;
        int retVal=-1;
        DocumentSectionMap::ConstIterator it = sections.begin();
        for (; it!= sections.end(); ++it)
        {
                if (pageIndex>=it.value().fromindex && 
pageIndex<=it.value().toindex)
                {
-                       found=true;
                        retVal=it.key();
                        break;
                }

Modified: branches/Version14x/Scribus/scribus/smlinestyle.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=24096&path=/branches/Version14x/Scribus/scribus/smlinestyle.cpp
==============================================================================
--- branches/Version14x/Scribus/scribus/smlinestyle.cpp (original)
+++ branches/Version14x/Scribus/scribus/smlinestyle.cpp Sat Oct 24 16:23:13 2020
@@ -378,7 +378,6 @@
 
 void SMLineStyle::unitChange()
 {
-       double oldRatio = unitRatio_;
        unitRatio_ = doc_->unitRatio();
        if (widget_)
                widget_->unitChange(doc_->unitIndex());


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

Reply via email to