fix #1523

jef · jef · commit 00efc2d74883 · 2009-02-01T16:59:03.000Z
git-svn-id: http://svn.osgeo.org/qgis/trunk/qgis@10084 c8812cc2-4d05-0410-92ff-de0c093fc19c
diff --git a/src/app/composer/qgscomposer.cpp b/src/app/composer/qgscomposer.cpp
@@ -587,7 +587,7 @@ void QgsComposer::on_mActionExportAsImage_activated( void )
   }
 
   myOutputFileNameQString = myQFileDialog->selectedFiles().last();
-  qWarning( myOutputFileNameQString.toLocal8Bit().data() );
+  qWarning( "%s", myOutputFileNameQString.toLocal8Bit().constData() );
   QString myFilterString = myQFileDialog->selectedFilter();
   QgsDebugMsg( QString( "Selected filter: %1" ).arg( myFilterString ) );
   QgsDebugMsg( QString( "Image type: %1" ).arg( myFilterMap[myFilterString] ) );
diff --git a/src/app/qgsabout.cpp b/src/app/qgsabout.cpp
@@ -56,8 +56,8 @@ void QgsAbout::init()
 
   QFile file( QgsApplication::authorsFilePath() );
 #ifdef QGISDEBUG
-  printf(( "Reading authors file " + file.fileName() +
-           ".............................................\n" ).toLocal8Bit().constData() );
+  printf( "Reading authors file %s.............................................\n",
+          file.fileName().toLocal8Bit().constData() );
 #endif
   if ( file.open( QIODevice::ReadOnly ) )
   {
@@ -103,8 +103,8 @@ void QgsAbout::init()
   // read the SPONSORS file and populate the text widget
   QFile sponsorFile( QgsApplication::sponsorsFilePath() );
 #ifdef QGISDEBUG
-  printf(( "Reading sponsors file " + sponsorFile.fileName() +
-           ".............................................\n" ).toLocal8Bit().constData() );
+  printf( "Reading sponsors file %s.............................................\n",
+          sponsorFile.fileName().toLocal8Bit().constData() );
 #endif
   if ( sponsorFile.open( QIODevice::ReadOnly ) )
   {
diff --git a/src/app/qgsdbsourceselect.cpp b/src/app/qgsdbsourceselect.cpp
@@ -442,8 +442,7 @@ void QgsDbSourceSelect::on_btnConnect_clicked()
     }
     else
     {
-      qDebug( "Unable to get list of spatially enabled tables from the database" );
-      qDebug( PQerrorMessage( pd ) );
+      qDebug( "Unable to get list of spatially enabled tables from the database\n%s", PQerrorMessage( pd ) );
     }
     // BEGIN CHANGES ECOS
     if ( cmbConnections->count() > 0 )
@@ -510,7 +509,7 @@ void QgsDbSourceSelect::setSql( const QModelIndex& index )
   QString schemaName = mTableModel.itemFromIndex( mProxyModel.mapToSource( schemaSibling ) )->text();
   QString tableName = mTableModel.itemFromIndex( mProxyModel.mapToSource( tableSibling ) )->text();
   QString tableString = "\"" + schemaName + "\".\"" + tableName + "\"";
-  qWarning( tableString.toUtf8() );
+  qWarning( "%s", tableString.toUtf8().constData() );
 
   QString currentSql;
   QModelIndex sqlSibling = index.sibling( index.row(), 4 );
diff --git a/src/app/qgsgeomtypedialog.cpp b/src/app/qgsgeomtypedialog.cpp
@@ -96,7 +96,7 @@ void QgsGeomTypeDialog::attributes( std::list<std::pair<QString, QString> >& at
     QTreeWidgetItem *item = *it;
     at.push_back( std::make_pair( item->text( 0 ), item->text( 1 ) ) );
 #ifdef QGISDEBUG
-    qWarning(( "appending " + item->text( 0 ) + "//" + item->text( 1 ) ).toLocal8Bit().data() );
+    qWarning( "appending %s//%s", item->text( 0 ).toLocal8Bit().constData(), item->text( 1 ).toLocal8Bit().constData() );
 #endif
     ++it;
   }
diff --git a/src/core/composer/qgscomposition.cpp b/src/core/composer/qgscomposition.cpp
@@ -232,7 +232,7 @@ void QgsComposition::addItemToZList( QgsComposerItem* item )
     return;
   }
   mItemZList.push_back( item );
-  qWarning( QString::number( mItemZList.size() ).toLocal8Bit().data() );
+  qWarning( "%d", mItemZList.size() );
   item->setZValue( mItemZList.size() );
 }
 
@@ -515,7 +515,7 @@ void QgsComposition::updateZValues()
     currentItem = *it;
     if ( currentItem )
     {
-      qWarning( QString::number( counter ).toLocal8Bit().data() );
+      qWarning( "%d", counter );
       currentItem->setZValue( counter );
     }
     ++counter;
@@ -529,7 +529,7 @@ void QgsComposition::sortZList()
   QLinkedList<QgsComposerItem*>::iterator before_it = mItemZList.begin();
   for ( ; before_it != mItemZList.end(); ++before_it )
   {
-    qWarning( QString::number(( *before_it )->zValue() ).toLocal8Bit().data() );
+    qWarning( "%lf", ( *before_it )->zValue() );
   }
 
   QMutableLinkedListIterator<QgsComposerItem*> it( mItemZList );
@@ -583,7 +583,7 @@ void QgsComposition::sortZList()
   QLinkedList<QgsComposerItem*>::iterator after_it = mItemZList.begin();
   for ( ; after_it != mItemZList.end(); ++after_it )
   {
-    qWarning( QString::number(( *after_it )->zValue() ).toLocal8Bit().data() );
+    qWarning( "%lf", ( *after_it )->zValue() );
   }
 }
 
diff --git a/src/core/qgslogger.cpp b/src/core/qgslogger.cpp
@@ -35,22 +35,22 @@ void QgsLogger::debug( const QString& msg, int debuglevel, const char* file, con
   {
     if ( file == NULL )
     {
-      qDebug( msg.toLocal8Bit().data() );
+      qDebug( "%s", msg.toLocal8Bit().constData() );
     }
     else if ( function == NULL )
     {
-      qDebug( "%s: %s", file, msg.toLocal8Bit().data() );
+      qDebug( "%s: %s", file, msg.toLocal8Bit().constData() );
     }
     else if ( line == -1 )
     {
-      qDebug( "%s: (%s) %s", file, function, msg.toLocal8Bit().data() );
+      qDebug( "%s: (%s) %s", file, function, msg.toLocal8Bit().constData() );
     }
     else
     {
 #ifndef _MSC_VER
-      qDebug( "%s: %d: (%s) %s", file, line, function, msg.toLocal8Bit().data() );
+      qDebug( "%s: %d: (%s) %s", file, line, function, msg.toLocal8Bit().constData() );
 #else
-      qDebug( "%s(%d) : (%s) %s", file, line, function, msg.toLocal8Bit().data() );
+      qDebug( "%s(%d) : (%s) %s", file, line, function, msg.toLocal8Bit().constData() );
 #endif
     }
   }
@@ -72,22 +72,22 @@ void QgsLogger::debug( const QString& var, int val, int debuglevel, const char*
   {
     if ( file == NULL )
     {
-      qDebug( "%s: %d", var.toLocal8Bit().data(), val );
+      qDebug( "%s: %d", var.toLocal8Bit().constData(), val );
     }
     else if ( function == NULL )
     {
-      qDebug( "%s: %s: %d", file, var.toLocal8Bit().data(), val );
+      qDebug( "%s: %s: %d", file, var.toLocal8Bit().constData(), val );
     }
     else if ( line == -1 )
     {
-      qDebug( "%s: (%s): %s: %d", file, function, var.toLocal8Bit().data(), val );
+      qDebug( "%s: (%s): %s: %d", file, function, var.toLocal8Bit().constData(), val );
     }
     else
     {
 #ifdef _MSC_VER
-      qDebug( "%s(%d): (%s), %s: %d", file, line, function, var.toLocal8Bit().data(), val );
+      qDebug( "%s(%d): (%s), %s: %d", file, line, function, var.toLocal8Bit().constData(), val );
 #else
-      qDebug( "%s: %d: (%s), %s: %d", file, line, function, var.toLocal8Bit().data(), val );
+      qDebug( "%s: %d: (%s), %s: %d", file, line, function, var.toLocal8Bit().constData(), val );
 #endif
     }
   }
@@ -109,40 +109,40 @@ void QgsLogger::debug( const QString& var, double val, int debuglevel, const cha
   {
     if ( file == NULL )
     {
-      qDebug( "%s: %f", var.toLocal8Bit().data(), val );
+      qDebug( "%s: %f", var.toLocal8Bit().constData(), val );
     }
     else if ( function == NULL )
     {
-      qDebug( "%s: %s: %f", file, var.toLocal8Bit().data(), val );
+      qDebug( "%s: %s: %f", file, var.toLocal8Bit().constData(), val );
     }
     else if ( line == -1 )
     {
-      qDebug( "%s: (%s): %s: %f", file, function, var.toLocal8Bit().data(), val );
+      qDebug( "%s: (%s): %s: %f", file, function, var.toLocal8Bit().constData(), val );
     }
     else
     {
 #ifdef _MSC_VER
-      qDebug( "%s(%d): (%s), %s: %f", file, line, function, var.toLocal8Bit().data(), val );
+      qDebug( "%s(%d): (%s), %s: %f", file, line, function, var.toLocal8Bit().constData(), val );
 #else
-      qDebug( "%s: %d: (%s), %s: %f", file, line, function, var.toLocal8Bit().data(), val );
+      qDebug( "%s: %d: (%s), %s: %f", file, line, function, var.toLocal8Bit().constData(), val );
 #endif
     }
   }
 }
 
 void QgsLogger::warning( const QString& msg )
 {
-  qWarning( msg.toLocal8Bit().data() );
+  qWarning( "%s", msg.toLocal8Bit().constData() );
 }
 
 void QgsLogger::critical( const QString& msg )
 {
-  qCritical( msg.toLocal8Bit().data() );
+  qCritical( "%s", msg.toLocal8Bit().constData() );
 }
 
 void QgsLogger::fatal( const QString& msg )
 {
-  qFatal( msg.toLocal8Bit().data() );
+  qFatal( "%s", msg.toLocal8Bit().constData() );
 }
 
 int QgsLogger::debugLevel()
diff --git a/src/core/qgsproject.cpp b/src/core/qgsproject.cpp
@@ -769,7 +769,7 @@ bool QgsProject::read()
     QString errorString = tr( "Project file read error: %1 at line %2 column %3" )
                           .arg( errorMsg ).arg( line ).arg( column );
 
-    qDebug( errorString.toUtf8().constData() );
+    qDebug( "%s", errorString.toUtf8().constData() );
 
     imp_->file.close();
 
diff --git a/src/core/symbology/qgssymbologyutils.cpp b/src/core/symbology/qgssymbologyutils.cpp
@@ -873,7 +873,7 @@ Qt::BrushStyle QgsSymbologyUtils::qString2BrushStyle( QString brushString )
   }
   else                        //return a null string
   {
-    qWarning( "Brush style \"" + brushString.toUtf8() + "\" not found in qString2BrushStyle" );
+    qWarning( "Brush style \"%s\" not found in qString2BrushStyle", brushString.toUtf8().constData() );
     return Qt::NoBrush;
   }
 }
diff --git a/src/gui/qgsprojectionselector.cpp b/src/gui/qgsprojectionselector.cpp
@@ -803,8 +803,7 @@ void QgsProjectionSelector::on_pbnFind_clicked()
   myFileInfo.setFile( myDatabaseFileName );
   if ( !myFileInfo.exists( ) ) //its not critical if this happens
   {
-    qDebug( myDatabaseFileName.toUtf8() );
-    qDebug( "User db does not exist" );
+    qDebug( "%s\nUser db does not exist", myDatabaseFileName.toUtf8().constData() );
     return ;
   }
   myResult = sqlite3_open( myDatabaseFileName.toUtf8().data(), &myDatabase );
diff --git a/src/gui/qgsquickprint.cpp b/src/gui/qgsquickprint.cpp
@@ -83,12 +83,12 @@ void QgsQuickPrint::setNorthArrow( QString theFileName )
 void QgsQuickPrint::setLogo1( QString theFileName )
 {
   mLogo1File = theFileName;
-  qDebug( "Logo1 set to: " + mLogo1File.toLocal8Bit() );
+  qDebug( "Logo1 set to: %s", mLogo1File.toLocal8Bit().constData() );
 }
 void QgsQuickPrint::setLogo2( QString theFileName )
 {
   mLogo2File = theFileName;
-  qDebug( "Logo2 set to: " + mLogo2File.toLocal8Bit() );
+  qDebug( "Logo2 set to: %s", mLogo2File.toLocal8Bit().constData() );
 }
 void QgsQuickPrint::setOutputPdf( QString theFileName )
 {
@@ -140,7 +140,7 @@ void QgsQuickPrint::printMap()
   int myPrintResolutionDpi = 300;
   myPrinter.setResolution( myPrintResolutionDpi );
   myPrinter.setOutputFormat( QPrinter::PdfFormat );
-  qDebug( "Printing to page size"  +  pageSizeToString( mPageSize ).toLocal8Bit() );
+  qDebug( "Printing to page size %s", pageSizeToString( mPageSize ).toLocal8Bit().constData() );
   myPrinter.setPageSize( mPageSize );
   myPrinter.setOutputFileName( mOutputFileName );
   myPrinter.setOrientation( QPrinter::Landscape );
@@ -588,8 +588,7 @@ void QgsQuickPrint::printMap()
   int myLogoXDim = ( myDrawableWidth / 100 ) * myLogoWidthPercent;
   int myLogoYDim = ( myDrawableHeight / 100 ) * myLogoHeightPercent;
   QPixmap myLogo1;
-  qDebug( "Logo1:" );
-  qDebug( mLogo1File.toLocal8Bit() );
+  qDebug( "Logo1: %s", mLogo1File.toLocal8Bit().constData() );
   myLogo1.fill( Qt::white );
   myLogo1.load( mLogo1File );
   myLogo1 = myLogo1.scaled( myLogoXDim, myLogoYDim, Qt::KeepAspectRatio );
@@ -1042,9 +1041,7 @@ QStringList QgsQuickPrint::wordWrap( QString theString,
     myList << myCumulativeLine.trimmed();
   }
 
-  //qDebug("Wrapped legend entry:");
-  //qDebug(theString);
-  //qDebug(myList.join("\n").toLocal8Bit());
+  //qDebug("Wrapped legend entry: %s\n%s", theString, myList.join("\n").toLocal8Bit().constData() );
   return myList;
 
 }
diff --git a/src/plugins/quick_print/quickprintgui.cpp b/src/plugins/quick_print/quickprintgui.cpp
@@ -146,7 +146,7 @@ void QuickPrintGui::on_buttonBox_accepted()
       myQuickPrint.setOutputPdf( myOutputFileName );
       QString myPageSizeString = cboPageSize->itemData( cboPageSize->currentIndex() ).toString();
       myQuickPrint.setPageSize( QgsQuickPrint::stringToPageSize( myPageSizeString ) );
-      qDebug( "Page size : " + myPageSizeString.toLocal8Bit() );
+      qDebug( "Page size : %s", myPageSizeString.toLocal8Bit().constData() );
       myQuickPrint.printMap();
     }
     else
diff --git a/src/plugins/wfs/qgswfssourceselect.cpp b/src/plugins/wfs/qgswfssourceselect.cpp
@@ -214,7 +214,7 @@ int QgsWFSSourceSelect::getCapabilitiesGET( QString uri, std::list<QString>& typ
 
   //print out result for a test
   QString resultString( result );
-  qWarning( resultString.toUtf8() );
+  qWarning( "%s", resultString.toUtf8().constData() );
 
   return 0;
 }
@@ -269,7 +269,7 @@ void QgsWFSSourceSelect::connectToServer()
   QSettings settings;
   QString key = "/Qgis/connections-wfs/" + cmbConnections->currentText() + "/url";
   mUri = settings.value( key ).toString();
-  qWarning( "url is: " + mUri.toUtf8() );
+  qWarning( "url is: %s", mUri.toUtf8().constData() );
 
   //make a GetCapabilities request
   std::list<QString> typenames;
@@ -349,7 +349,7 @@ void QgsWFSSourceSelect::addLayer()
   {
     uri.append( "?" );
   }
-  qWarning( uri.toUtf8() + "SERVICE=WFS&VERSION=1.0.0&REQUEST=GetFeature&TYPENAME=" + typeName.toUtf8() );
+  qWarning( "%sSERVICE=WFS&VERSION=1.0.0&REQUEST=GetFeature&TYPENAME=%s", uri.toUtf8().constData(), typeName.toUtf8().constData() );
 
   //get CRS
   QString crsString;
@@ -385,7 +385,7 @@ void QgsWFSSourceSelect::changeCRSFilter()
   if ( currentTreeItem )
   {
     QString currentTypename = currentTreeItem->text( 1 );
-    qWarning( "the current typename is: " + currentTypename.toUtf8() );
+    qWarning( "the current typename is: %s", currentTypename.toUtf8().constData() );
 
     std::map<QString, std::list<QString> >::const_iterator crsIterator = mAvailableCRS.find( currentTypename );
     if ( crsIterator != mAvailableCRS.end() )
diff --git a/src/providers/postgres/qgspostgresextentthread.cpp b/src/providers/postgres/qgspostgresextentthread.cpp
@@ -98,10 +98,7 @@ void QgsPostgresExtentThread::run()
         "ymax(extent(" + geometryColumn + ")) as ymax," "ymin(extent(" + geometryColumn + ")) as ymin" " from " + tableName;
 #endif
 
-#ifdef QGISDEBUG
-  qDebug( "+++++++++QgsPostgresExtentThread::run -  Getting extents using schema.table: " + sql.toUtf8() );
-#endif
-
+  QgsDebugMsg( QString( "Getting extents using schema.table: %1" ).arg( sql ) );
 
   QgsDebugMsg( "About to issue query." );
 
diff --git a/src/providers/wfs/qgswfsdata.cpp b/src/providers/wfs/qgswfsdata.cpp
@@ -57,8 +57,7 @@ QgsWFSData::QgsWFSData(
     if ( it->startsWith( "TYPENAME", Qt::CaseInsensitive ) )
     {
       mTypeName = it->section( "=", 1, 1 );
-      qWarning( "mTypeName is:" );
-      qWarning( mTypeName.toLocal8Bit().data() );
+      qWarning( "mTypeName is: %s", mTypeName.toLocal8Bit().constData() );
     }
   }
 
diff --git a/src/providers/wfs/qgswfsprovider.cpp b/src/providers/wfs/qgswfsprovider.cpp
@@ -296,10 +296,8 @@ int QgsWFSProvider::getFeatureGET( const QString& uri, const QString& geometryAt
     return 1;
   }
 
-  qWarning( "feature count after request is:" );
-  qWarning( QString::number( mFeatures.size() ).toLocal8Bit().data() );
-  qWarning( "mExtent after request is:" );
-  qWarning( mExtent.toString().toLocal8Bit().data() );
+  qWarning( "feature count after request is: %d", mFeatures.size() );
+  qWarning( "mExtent after request is: %s",  mExtent.toString().toLocal8Bit().data() );
 
   for ( QList<QgsFeature*>::iterator it = mFeatures.begin(); it != mFeatures.end(); ++it )
   {
diff --git a/tests/src/core/qgsrenderchecker.cpp b/tests/src/core/qgsrenderchecker.cpp
@@ -138,14 +138,12 @@ bool QgsRenderChecker::compareImages( QString theTestName )
   // Put the same info to debug too
   //
 
-  qDebug( "Expected size: " + QString::number( myExpectedImage.width() ).toLocal8Bit() + "w x " +
-          QString::number( myExpectedImage.width() ).toLocal8Bit() + "h" );
-  qDebug( "Actual   size: " + QString::number( myResultImage.width() ).toLocal8Bit() + "w x " +
-          QString::number( myResultImage.width() ).toLocal8Bit() + "h" );
+  qDebug( "Expected size: %dw x %dh", myExpectedImage.width(), myExpectedImage.height() );
+  qDebug( "Actual   size: %dw x %dh", myResultImage.width(), myResultImage.height() );
 
   if ( mMatchTarget != myPixelCount )
   {
-    qDebug( QString( "Test image and result image for %1 are different - FAILING!" ).arg( theTestName ).toLocal8Bit() );
+    qDebug( "Test image and result image for %s are different - FAILING!", theTestName.toLocal8Bit().constData() );
     mReport += "<tr><td colspan=3>";
     mReport += "<font color=red>Expected image and result image for " + theTestName + " are different dimensions - FAILING!</font>";
     mReport += "</td></tr>";
@@ -180,9 +178,7 @@ bool QgsRenderChecker::compareImages( QString theTestName )
   //
   // Send match result to debug
   //
-  qDebug( QString::number( mMismatchCount ).toLocal8Bit() + "/" +
-          QString::number( mMatchTarget ).toLocal8Bit() +
-          " pixels mismatched" );;
+  qDebug( "%d/%d pixels mismatched", mMismatchCount, mMatchTarget );
 
   //
   // Send match result to report
diff --git a/tests/src/core/regression1141.cpp b/tests/src/core/regression1141.cpp
@@ -99,8 +99,7 @@ void Regression1141::diacriticalTest()
   mFields.insert( 0, myField );
   mCRS = QgsCoordinateReferenceSystem( GEOWkt );
 
-  qDebug( "Checking test dataset exists..." );
-  qDebug( mFileName.toLocal8Bit() );
+  qDebug( "Checking test dataset exists...\n%s", mFileName.toLocal8Bit().constData() );
 
   if ( !QFile::exists( mFileName ) )
   {
diff --git a/tests/src/core/testqgsapplication.cpp b/tests/src/core/testqgsapplication.cpp
diff --git a/tests/src/core/testqgsgeometry.cpp b/tests/src/core/testqgsgeometry.cpp
diff --git a/tests/src/core/testqgsmaprenderer.cpp b/tests/src/core/testqgsmaprenderer.cpp
diff --git a/tests/src/core/testqgsrasterlayer.cpp b/tests/src/core/testqgsrasterlayer.cpp
diff --git a/tests/src/core/testqgsrenderers.cpp b/tests/src/core/testqgsrenderers.cpp

Original file line number	Diff line number	Diff line change
`@@ -587,7 +587,7 @@ void QgsComposer::on_mActionExportAsImage_activated( void )`
`587`	`587`	`}`
`588`	`588`
`589`	`589`	`myOutputFileNameQString = myQFileDialog->selectedFiles().last();`
`590`		`- qWarning( myOutputFileNameQString.toLocal8Bit().data() );`
	`590`	`+ qWarning( "%s", myOutputFileNameQString.toLocal8Bit().constData() );`
`591`	`591`	`QString myFilterString = myQFileDialog->selectedFilter();`
`592`	`592`	`QgsDebugMsg( QString( "Selected filter: %1" ).arg( myFilterString ) );`
`593`	`593`	`QgsDebugMsg( QString( "Image type: %1" ).arg( myFilterMap[myFilterString] ) );`
Original file line number	Diff line number	Diff line change
`@@ -442,8 +442,7 @@ void QgsDbSourceSelect::on_btnConnect_clicked()`
`442`	`442`	`}`
`443`	`443`	`else`
`444`	`444`	`{`
`445`		`- qDebug( "Unable to get list of spatially enabled tables from the database" );`
`446`		`- qDebug( PQerrorMessage( pd ) );`
	`445`	`+ qDebug( "Unable to get list of spatially enabled tables from the database\n%s", PQerrorMessage( pd ) );`
`447`	`446`	`}`
`448`	`447`	`// BEGIN CHANGES ECOS`
`449`	`448`	`if ( cmbConnections->count() > 0 )`
`@@ -510,7 +509,7 @@ void QgsDbSourceSelect::setSql( const QModelIndex& index )`
`510`	`509`	`QString schemaName = mTableModel.itemFromIndex( mProxyModel.mapToSource( schemaSibling ) )->text();`
`511`	`510`	`QString tableName = mTableModel.itemFromIndex( mProxyModel.mapToSource( tableSibling ) )->text();`
`512`	`511`	`QString tableString = "\"" + schemaName + "\".\"" + tableName + "\"";`
`513`		`- qWarning( tableString.toUtf8() );`
	`512`	`+ qWarning( "%s", tableString.toUtf8().constData() );`
`514`	`513`
`515`	`514`	`QString currentSql;`
`516`	`515`	`QModelIndex sqlSibling = index.sibling( index.row(), 4 );`
Original file line number	Diff line number	Diff line change
`@@ -96,7 +96,7 @@ void QgsGeomTypeDialog::attributes( std::list<std::pair<QString, QString> >& at`
`96`	`96`	`QTreeWidgetItem item = it;`
`97`	`97`	`at.push_back( std::make_pair( item->text( 0 ), item->text( 1 ) ) );`
`98`	`98`	`#ifdef QGISDEBUG`
`99`		`- qWarning(( "appending " + item->text( 0 ) + "//" + item->text( 1 ) ).toLocal8Bit().data() );`
	`99`	`+ qWarning( "appending %s//%s", item->text( 0 ).toLocal8Bit().constData(), item->text( 1 ).toLocal8Bit().constData() );`
`100`	`100`	`#endif`
`101`	`101`	`++it;`
`102`	`102`	`}`
Original file line number	Diff line number	Diff line change
`@@ -232,7 +232,7 @@ void QgsComposition::addItemToZList( QgsComposerItem* item )`
`232`	`232`	`return;`
`233`	`233`	`}`
`234`	`234`	`mItemZList.push_back( item );`
`235`		`- qWarning( QString::number( mItemZList.size() ).toLocal8Bit().data() );`
	`235`	`+ qWarning( "%d", mItemZList.size() );`
`236`	`236`	`item->setZValue( mItemZList.size() );`
`237`	`237`	`}`
`238`	`238`
`@@ -515,7 +515,7 @@ void QgsComposition::updateZValues()`
`515`	`515`	`currentItem = *it;`
`516`	`516`	`if ( currentItem )`
`517`	`517`	`{`
`518`		`- qWarning( QString::number( counter ).toLocal8Bit().data() );`
	`518`	`+ qWarning( "%d", counter );`
`519`	`519`	`currentItem->setZValue( counter );`
`520`	`520`	`}`
`521`	`521`	`++counter;`
`@@ -529,7 +529,7 @@ void QgsComposition::sortZList()`
`529`	`529`	`QLinkedList<QgsComposerItem*>::iterator before_it = mItemZList.begin();`
`530`	`530`	`for ( ; before_it != mItemZList.end(); ++before_it )`
`531`	`531`	`{`
`532`		`- qWarning( QString::number(( *before_it )->zValue() ).toLocal8Bit().data() );`
	`532`	`+ qWarning( "%lf", ( *before_it )->zValue() );`
`533`	`533`	`}`
`534`	`534`
`535`	`535`	`QMutableLinkedListIterator<QgsComposerItem*> it( mItemZList );`
`@@ -583,7 +583,7 @@ void QgsComposition::sortZList()`
`583`	`583`	`QLinkedList<QgsComposerItem*>::iterator after_it = mItemZList.begin();`
`584`	`584`	`for ( ; after_it != mItemZList.end(); ++after_it )`
`585`	`585`	`{`
`586`		`- qWarning( QString::number(( *after_it )->zValue() ).toLocal8Bit().data() );`
	`586`	`+ qWarning( "%lf", ( *after_it )->zValue() );`
`587`	`587`	`}`
`588`	`588`	`}`
`589`	`589`
Original file line number	Diff line number	Diff line change
`@@ -35,22 +35,22 @@ void QgsLogger::debug( const QString& msg, int debuglevel, const char* file, con`
`35`	`35`	`{`
`36`	`36`	`if ( file == NULL )`
`37`	`37`	`{`
`38`		`- qDebug( msg.toLocal8Bit().data() );`
	`38`	`+ qDebug( "%s", msg.toLocal8Bit().constData() );`
`39`	`39`	`}`
`40`	`40`	`else if ( function == NULL )`
`41`	`41`	`{`
`42`		`- qDebug( "%s: %s", file, msg.toLocal8Bit().data() );`
	`42`	`+ qDebug( "%s: %s", file, msg.toLocal8Bit().constData() );`
`43`	`43`	`}`
`44`	`44`	`else if ( line == -1 )`
`45`	`45`	`{`
`46`		`- qDebug( "%s: (%s) %s", file, function, msg.toLocal8Bit().data() );`
	`46`	`+ qDebug( "%s: (%s) %s", file, function, msg.toLocal8Bit().constData() );`
`47`	`47`	`}`
`48`	`48`	`else`
`49`	`49`	`{`
`50`	`50`	`#ifndef _MSC_VER`
`51`		`- qDebug( "%s: %d: (%s) %s", file, line, function, msg.toLocal8Bit().data() );`
	`51`	`+ qDebug( "%s: %d: (%s) %s", file, line, function, msg.toLocal8Bit().constData() );`
`52`	`52`	`#else`
`53`		`- qDebug( "%s(%d) : (%s) %s", file, line, function, msg.toLocal8Bit().data() );`
	`53`	`+ qDebug( "%s(%d) : (%s) %s", file, line, function, msg.toLocal8Bit().constData() );`
`54`	`54`	`#endif`
`55`	`55`	`}`
`56`	`56`	`}`
`@@ -72,22 +72,22 @@ void QgsLogger::debug( const QString& var, int val, int debuglevel, const char*`
`72`	`72`	`{`
`73`	`73`	`if ( file == NULL )`
`74`	`74`	`{`
`75`		`- qDebug( "%s: %d", var.toLocal8Bit().data(), val );`
	`75`	`+ qDebug( "%s: %d", var.toLocal8Bit().constData(), val );`
`76`	`76`	`}`
`77`	`77`	`else if ( function == NULL )`
`78`	`78`	`{`
`79`		`- qDebug( "%s: %s: %d", file, var.toLocal8Bit().data(), val );`
	`79`	`+ qDebug( "%s: %s: %d", file, var.toLocal8Bit().constData(), val );`
`80`	`80`	`}`
`81`	`81`	`else if ( line == -1 )`
`82`	`82`	`{`
`83`		`- qDebug( "%s: (%s): %s: %d", file, function, var.toLocal8Bit().data(), val );`
	`83`	`+ qDebug( "%s: (%s): %s: %d", file, function, var.toLocal8Bit().constData(), val );`
`84`	`84`	`}`
`85`	`85`	`else`
`86`	`86`	`{`
`87`	`87`	`#ifdef _MSC_VER`
`88`		`- qDebug( "%s(%d): (%s), %s: %d", file, line, function, var.toLocal8Bit().data(), val );`
	`88`	`+ qDebug( "%s(%d): (%s), %s: %d", file, line, function, var.toLocal8Bit().constData(), val );`
`89`	`89`	`#else`
`90`		`- qDebug( "%s: %d: (%s), %s: %d", file, line, function, var.toLocal8Bit().data(), val );`
	`90`	`+ qDebug( "%s: %d: (%s), %s: %d", file, line, function, var.toLocal8Bit().constData(), val );`
`91`	`91`	`#endif`
`92`	`92`	`}`
`93`	`93`	`}`
`@@ -109,40 +109,40 @@ void QgsLogger::debug( const QString& var, double val, int debuglevel, const cha`
`109`	`109`	`{`
`110`	`110`	`if ( file == NULL )`
`111`	`111`	`{`
`112`		`- qDebug( "%s: %f", var.toLocal8Bit().data(), val );`
	`112`	`+ qDebug( "%s: %f", var.toLocal8Bit().constData(), val );`
`113`	`113`	`}`
`114`	`114`	`else if ( function == NULL )`
`115`	`115`	`{`
`116`		`- qDebug( "%s: %s: %f", file, var.toLocal8Bit().data(), val );`
	`116`	`+ qDebug( "%s: %s: %f", file, var.toLocal8Bit().constData(), val );`
`117`	`117`	`}`
`118`	`118`	`else if ( line == -1 )`
`119`	`119`	`{`
`120`		`- qDebug( "%s: (%s): %s: %f", file, function, var.toLocal8Bit().data(), val );`
	`120`	`+ qDebug( "%s: (%s): %s: %f", file, function, var.toLocal8Bit().constData(), val );`
`121`	`121`	`}`
`122`	`122`	`else`
`123`	`123`	`{`
`124`	`124`	`#ifdef _MSC_VER`
`125`		`- qDebug( "%s(%d): (%s), %s: %f", file, line, function, var.toLocal8Bit().data(), val );`
	`125`	`+ qDebug( "%s(%d): (%s), %s: %f", file, line, function, var.toLocal8Bit().constData(), val );`
`126`	`126`	`#else`
`127`		`- qDebug( "%s: %d: (%s), %s: %f", file, line, function, var.toLocal8Bit().data(), val );`
	`127`	`+ qDebug( "%s: %d: (%s), %s: %f", file, line, function, var.toLocal8Bit().constData(), val );`
`128`	`128`	`#endif`
`129`	`129`	`}`
`130`	`130`	`}`
`131`	`131`	`}`
`132`	`132`
`133`	`133`	`void QgsLogger::warning( const QString& msg )`
`134`	`134`	`{`
`135`		`- qWarning( msg.toLocal8Bit().data() );`
	`135`	`+ qWarning( "%s", msg.toLocal8Bit().constData() );`
`136`	`136`	`}`
`137`	`137`
`138`	`138`	`void QgsLogger::critical( const QString& msg )`
`139`	`139`	`{`
`140`		`- qCritical( msg.toLocal8Bit().data() );`
	`140`	`+ qCritical( "%s", msg.toLocal8Bit().constData() );`
`141`	`141`	`}`
`142`	`142`
`143`	`143`	`void QgsLogger::fatal( const QString& msg )`
`144`	`144`	`{`
`145`		`- qFatal( msg.toLocal8Bit().data() );`
	`145`	`+ qFatal( "%s", msg.toLocal8Bit().constData() );`
`146`	`146`	`}`
`147`	`147`
`148`	`148`	`int QgsLogger::debugLevel()`
Original file line number	Diff line number	Diff line change
`@@ -873,7 +873,7 @@ Qt::BrushStyle QgsSymbologyUtils::qString2BrushStyle( QString brushString )`
`873`	`873`	`}`
`874`	`874`	`else //return a null string`
`875`	`875`	`{`
`876`		`- qWarning( "Brush style \"" + brushString.toUtf8() + "\" not found in qString2BrushStyle" );`
	`876`	`+ qWarning( "Brush style \"%s\" not found in qString2BrushStyle", brushString.toUtf8().constData() );`
`877`	`877`	`return Qt::NoBrush;`
`878`	`878`	`}`
`879`	`879`	`}`
Original file line number	Diff line number	Diff line change
`@@ -803,8 +803,7 @@ void QgsProjectionSelector::on_pbnFind_clicked()`
`803`	`803`	`myFileInfo.setFile( myDatabaseFileName );`
`804`	`804`	`if ( !myFileInfo.exists( ) ) //its not critical if this happens`
`805`	`805`	`{`
`806`		`- qDebug( myDatabaseFileName.toUtf8() );`
`807`		`- qDebug( "User db does not exist" );`
	`806`	`+ qDebug( "%s\nUser db does not exist", myDatabaseFileName.toUtf8().constData() );`
`808`	`807`	`return ;`
`809`	`808`	`}`
`810`	`809`	`myResult = sqlite3_open( myDatabaseFileName.toUtf8().data(), &myDatabase );`
Original file line number	Diff line number	Diff line change
`@@ -146,7 +146,7 @@ void QuickPrintGui::on_buttonBox_accepted()`
`146`	`146`	`myQuickPrint.setOutputPdf( myOutputFileName );`
`147`	`147`	`QString myPageSizeString = cboPageSize->itemData( cboPageSize->currentIndex() ).toString();`
`148`	`148`	`myQuickPrint.setPageSize( QgsQuickPrint::stringToPageSize( myPageSizeString ) );`
`149`		`- qDebug( "Page size : " + myPageSizeString.toLocal8Bit() );`
	`149`	`+ qDebug( "Page size : %s", myPageSizeString.toLocal8Bit().constData() );`
`150`	`150`	`myQuickPrint.printMap();`
`151`	`151`	`}`
`152`	`152`	`else`
Original file line number	Diff line number	Diff line change
`@@ -57,8 +57,7 @@ QgsWFSData::QgsWFSData(`
`57`	`57`	`if ( it->startsWith( "TYPENAME", Qt::CaseInsensitive ) )`
`58`	`58`	`{`
`59`	`59`	`mTypeName = it->section( "=", 1, 1 );`
`60`		`- qWarning( "mTypeName is:" );`
`61`		`- qWarning( mTypeName.toLocal8Bit().data() );`
	`60`	`+ qWarning( "mTypeName is: %s", mTypeName.toLocal8Bit().constData() );`
`62`	`61`	`}`
`63`	`62`	`}`
`64`	`63`
Original file line number	Diff line number	Diff line change
`@@ -296,10 +296,8 @@ int QgsWFSProvider::getFeatureGET( const QString& uri, const QString& geometryAt`
`296`	`296`	`return 1;`
`297`	`297`	`}`
`298`	`298`
`299`		`- qWarning( "feature count after request is:" );`
`300`		`- qWarning( QString::number( mFeatures.size() ).toLocal8Bit().data() );`
`301`		`- qWarning( "mExtent after request is:" );`
`302`		`- qWarning( mExtent.toString().toLocal8Bit().data() );`
	`299`	`+ qWarning( "feature count after request is: %d", mFeatures.size() );`
	`300`	`+ qWarning( "mExtent after request is: %s", mExtent.toString().toLocal8Bit().data() );`
`303`	`301`
`304`	`302`	`for ( QList<QgsFeature*>::iterator it = mFeatures.begin(); it != mFeatures.end(); ++it )`
`305`	`303`	`{`