Revert qprocess code. Revisions reverted: 22026, 22030, 22044, 22048,

[lyx.git] / src / support / FileName.cpp

diff --git a/src/support/FileName.cpp b/src/support/FileName.cpp
index 2c8fa57b1e0fec6e986449ce3ddfc11d16d55b0f..40e717516c971614267e9dab8f06f3678b205061 100644 (file)
--- a/src/support/FileName.cpp
+++ b/src/support/FileName.cpp
@@ -25,6 +25,7 @@
 #include <QFile>
 #include <QFileInfo>
 #include <QList>
+#include <QTime>
 
 #include <boost/assert.hpp>
 
@@ -42,12 +43,7 @@
 #include <cerrno>
 #include <fcntl.h>
 
-
-using std::map;
-using std::string;
-using std::ifstream;
-using std::ostringstream;
-using std::endl;
+using namespace std;
 
 namespace lyx {
 namespace support {
@@ -292,11 +288,18 @@ unsigned long FileName::checksum() const
        }
        // a directory may be passed here so we need to test it. (bug 3622)
        if (isDirectory()) {
-               LYXERR0('\\' << absFilename() << "\" is a directory!");
+               LYXERR0('"' << absFilename() << "\" is a directory!");
                return 0;
        }
-       LYXERR0("Checksumming \"" << absFilename() << "\".");
-       return sum(absFilename().c_str());
+       if (!lyxerr.debugging(Debug::FILES))
+               return sum(absFilename().c_str());
+
+       QTime t;
+       t.start();
+       unsigned long r = sum(absFilename().c_str());
+       lyxerr << "Checksumming \"" << absFilename() << "\" lasted "
+               << t.elapsed() << " ms." << endl;
+       return r;
 }
 
 
@@ -304,8 +307,7 @@ bool FileName::removeFile() const
 {
        bool const success = QFile::remove(d->fi.absoluteFilePath());
        if (!success && exists())
-               lyxerr << "FileName::removeFile(): Could not delete file "
-                       << *this << "." << endl;
+               LYXERR0("Could not delete file " << *this);
        return success;
 }
 
@@ -314,32 +316,32 @@ static bool rmdir(QFileInfo const & fi)
 {
        QDir dir(fi.absoluteFilePath());
        QFileInfoList list = dir.entryInfoList();
-       bool global_success = true;
+       bool success = true;
        for (int i = 0; i != list.size(); ++i) {
                if (list.at(i).fileName() == ".")
                        continue;
                if (list.at(i).fileName() == "..")
                        continue;
-               bool success;
+               bool removed;
                if (list.at(i).isDir()) {
-                       LYXERR(Debug::FILES, "Erasing dir " 
+                       LYXERR(Debug::FILES, "Removing dir " 
                                << fromqstr(list.at(i).absoluteFilePath()));
-                       success = rmdir(list.at(i));
+                       removed = rmdir(list.at(i));
                }
                else {
-                       LYXERR(Debug::FILES, "Erasing file " 
+                       LYXERR(Debug::FILES, "Removing file " 
                                << fromqstr(list.at(i).absoluteFilePath()));
-                       success = dir.remove(list.at(i).fileName());
+                       removed = dir.remove(list.at(i).fileName());
                }
-               if (!success) {
-                       global_success = false;
-                       lyxerr << "Could not delete "
-                               << fromqstr(list.at(i).absoluteFilePath()) << "." << endl;
+               if (!removed) {
+                       success = false;
+                       LYXERR0("Could not delete "
+                               << fromqstr(list.at(i).absoluteFilePath()));
                }
        } 
        QDir parent = fi.absolutePath();
-       global_success |= parent.rmdir(fi.fileName());
-       return global_success;
+       success &= parent.rmdir(fi.fileName());
+       return success;
 }
 
 
@@ -347,7 +349,7 @@ bool FileName::destroyDirectory() const
 {
        bool const success = rmdir(d->fi);
        if (!success)
-               lyxerr << "Could not delete " << *this << "." << endl;
+               LYXERR0("Could not delete " << *this);
 
        return success;
 }
@@ -375,8 +377,7 @@ docstring FileName::displayName(int threshold) const
 docstring FileName::fileContents(string const & encoding) const
 {
        if (!isReadableFile()) {
-               LYXERR0("File '" << *this
-                       << "' is not redable!");
+               LYXERR0("File '" << *this << "' is not redable!");
                return docstring();
        }
 
@@ -457,8 +458,8 @@ string FileName::guessFormatFromContents() const
        // GZIP \037\213        http://www.ietf.org/rfc/rfc1952.txt
        // ZIP  PK...                   http://www.halyava.ru/document/ind_arch.htm
        // Z    \037\235                UNIX compress
-       // paranoia check
 
+       // paranoia check
        if (empty() || !isReadableFile())
                return string();