- ignore ENOENT and EAGAIN in renameFile (bug in system?)

- replace system-call "mkdir..." with QDir::mkdir

1 files changed, 8 insertions, 6 deletions

diff --git a/library/filemanager.cpp b/library/filemanager.cpp
index 2e5efdd..fbfa1d9 100644
--- a/library/filemanager.cpp
+++ b/library/filemanager.cpp
@@ -233,51 +233,54 @@ bool FileManager::copyFile( const QString & src, const QString & dest )
         {
             int bytesWritten = destFile.writeBlock( buffer, bytesRead );
             if ( bytesWritten < 0 )
                 ok = FALSE;
             else
                 bytesRead -= bytesWritten;
         }
     }
     srcFile.close();
     destFile.close();
     // Set file permissions
     struct stat status;
     if( stat( QFile::encodeName( src ), &status ) == 0 )
     {
         chmod( QFile::encodeName( dest ), status.st_mode );
     }
     return ok;
 }
 
 
 bool FileManager::renameFile( const QString & src, const QString & dest )
 {
     if( rename( QFile::encodeName( src ), QFile::encodeName( dest ) ) == -1);
     {
+        if ( errno != 2 && errno != 11 ) //ignore ENOENT and EAGAIN - bug in system?
+        {
         qWarning( "problem renaming file %s to %s, errno: %d", src.latin1(), dest.latin1(), errno );
         return false;
     }
+    }
     return true;
 }
 
 /*!
   Opens the document specified by \a f as a readable QIODevice.
   The caller must delete the return value.
 
   Returns 0 if the operation fails.
 */
 QIODevice* FileManager::openFile( const DocLnk& f )
 {
     QString fn = f.file();
     QFile* fl = new QFile( fn );
     if ( !fl->open( IO_ReadOnly ) )
     {
         delete fl;
         fl = 0;
     }
     return fl;
 }
 
 /*!
   Opens the document specified by \a f as a writable QIODevice.
   The caller must delete the return value.
@@ -290,38 +293,37 @@ QIODevice* FileManager::saveFile( const DocLnk& f )
     ensurePathExists( fn );
     QFile* fl = new QFile( fn );
     if ( fl->open( IO_WriteOnly ) )
     {
         f.writeLink();
     }
     else
     {
         delete fl;
         fl = 0;
     }
     return fl;
 }
 
 /*!
   Returns whether the document specified by \a f current exists
   as a file on disk.
 */
 bool FileManager::exists( const DocLnk &f )
 {
     return QFile::exists(f.file());
 }
 
 /*!
-  Ensures that the path \a fn exists, by creating required directories.
+  Ensures that the path \a fileName exists, by creating required directories.
   Returns TRUE if successful.
 */
-bool FileManager::ensurePathExists( const QString &fn )
+bool FileManager::ensurePathExists( const QString &fileName )
 {
-    QFileInfo fi(fn);
-    fi.setFile( fi.dirPath(TRUE) );
-    if ( !fi.exists() )
+    QDir directory = QFileInfo( fileName ).dir();
+    if ( !directory.exists() )
     {
-        if ( system( ("mkdir -p " + QFile::encodeName( fi.filePath() ) ) ) )
+        if ( !directory.mkdir( directory.absPath() ) )
             return FALSE;
     }
     return TRUE;
 }