[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdesupport/soprano/backends/sesame2
From:       Albert Astals Cid <tsdgeos () terra ! es>
Date:       2010-09-03 20:57:47
Message-ID: 20100903210148.A8126AC884 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1171427 by aacid:

const & in foreach


 M  +3 -3      sesame2backend.cpp  


--- trunk/kdesupport/soprano/backends/sesame2/sesame2backend.cpp #1171426:1171427
@@ -84,7 +84,7 @@
 
     // FIXME: support inference option
 
-    Q_FOREACH( BackendSetting s, settings ) {
+    Q_FOREACH( const BackendSetting& s, settings ) {
         if ( s.option() == BackendOptionUser ) {
             // no user options ATM
             qDebug() << "(Soprano::Sesame2::BackendPlugin) no user options \
supported."; @@ -138,7 +138,7 @@
 bool Soprano::Sesame2::BackendPlugin::deleteModelData( const BackendSettings& \
settings ) const  {
     QString path;
-    Q_FOREACH( BackendSetting s, settings ) {
+    Q_FOREACH( const BackendSetting& s, settings ) {
         if ( s.option() == BackendOptionStorageDir ) {
             path = s.value().toString();
             break;
@@ -153,7 +153,7 @@
     // FIXME: is there a way to get the actual list of files
     QDir dir( path );
     QStringList files = dir.entryList( QStringList() << "triples*" << "namespaces*" \
                << "values*", QDir::Files );
-    foreach( QString file, files ) {
+    foreach( const QString& file, files ) {
         if ( !dir.remove( file ) ) {
             setError( "Failed to remove file '" + dir.filePath( file ), \
Error::ErrorUnknown );  return false;


[prev in list] [next in list] [prev in thread] [next in thread]