SyncEngine: Adjust free space messaging
authorChristian Kamm <mail@ckamm.de>
Fri, 22 Sep 2017 09:06:29 +0000 (11:06 +0200)
committerRoeland Jago Douma <roeland@famdouma.nl>
Thu, 5 Oct 2017 20:01:35 +0000 (22:01 +0200)
Before, the way the info message was phrased implied a failure. And
since the large numbers can be hard to compare that can be confusing.

src/libsync/syncengine.cpp

index 22a8068b06ad585ed553a8ad09dce851d3d71137..48d439b769339c352b7d02e20377c51666bd6dd3 100644 (file)
@@ -762,9 +762,9 @@ void SyncEngine::startSync()
     const qint64 minFree = criticalFreeSpaceLimit();
     const qint64 freeBytes = Utility::freeDiskSpace(_localPath);
     if (freeBytes >= 0) {
-        qCInfo(lcEngine) << "There are" << freeBytes << "bytes available at" << _localPath
-                         << "and at least" << minFree << "are required";
         if (freeBytes < minFree) {
+            qCWarning(lcEngine()) << "Too little space available at" << _localPath << ". Have"
+                                  << freeBytes << "bytes and require at least" << minFree << "bytes";
             _anotherSyncNeeded = DelayedFollowUp;
             csyncError(tr("Only %1 are available, need at least %2 to start",
                 "Placeholders are postfixed with file sizes using Utility::octetsToString()")
@@ -773,6 +773,8 @@ void SyncEngine::startSync()
                                Utility::octetsToString(minFree)));
             finalize(false);
             return;
+        } else {
+            qCInfo(lcEngine) << "There are" << freeBytes << "bytes available at" << _localPath;
         }
     } else {
         qCWarning(lcEngine) << "Could not determine free space available at" << _localPath;