From eae3f7ce0acb163b50be32d93af88f2ea99c7282 Mon Sep 17 00:00:00 2001 From: Christian Kamm Date: Fri, 22 Sep 2017 11:06:29 +0200 Subject: [PATCH] SyncEngine: Adjust free space messaging 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 | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/libsync/syncengine.cpp b/src/libsync/syncengine.cpp index 22a8068b0..48d439b76 100644 --- a/src/libsync/syncengine.cpp +++ b/src/libsync/syncengine.cpp @@ -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;