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.
This commit is contained in:
Christian Kamm 2017-09-22 11:06:29 +02:00
parent aa322d41aa
commit eae3f7ce0a

View file

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