Skip to content

Commit f6c4c32

Browse files
claucambramgallien
authored andcommitted
Use QFileInfo::exists where we are only creating a QFileInfo to check if file exists, as it is faster
Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
1 parent 7f6b18b commit f6c4c32

2 files changed

Lines changed: 5 additions & 5 deletions

File tree

src/gui/tray/usermodel.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -493,24 +493,24 @@ void User::slotProgressInfo(const QString &folder, const ProgressInfo &progress)
493493
continue;
494494
}
495495

496-
if (activity._syncFileItemStatus == SyncFileItem::Conflict && !QFileInfo(f->path() + activity._file).exists()) {
496+
if (activity._syncFileItemStatus == SyncFileItem::Conflict && !QFileInfo::exists(f->path() + activity._file)) {
497497
_activityModel->removeActivityFromActivityList(activity);
498498
continue;
499499
}
500500

501-
if (activity._syncFileItemStatus == SyncFileItem::FileLocked && !QFileInfo(f->path() + activity._file).exists()) {
501+
if (activity._syncFileItemStatus == SyncFileItem::FileLocked && !QFileInfo::exists(f->path() + activity._file)) {
502502
_activityModel->removeActivityFromActivityList(activity);
503503
continue;
504504
}
505505

506506

507-
if (activity._syncFileItemStatus == SyncFileItem::FileIgnored && !QFileInfo(f->path() + activity._file).exists()) {
507+
if (activity._syncFileItemStatus == SyncFileItem::FileIgnored && !QFileInfo::exists(f->path() + activity._file)) {
508508
_activityModel->removeActivityFromActivityList(activity);
509509
continue;
510510
}
511511

512512

513-
if (!QFileInfo(f->path() + activity._file).exists()) {
513+
if (!QFileInfo::exists(f->path() + activity._file)) {
514514
_activityModel->removeActivityFromActivityList(activity);
515515
continue;
516516
}

src/libsync/syncengine.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -286,7 +286,7 @@ void SyncEngine::conflictRecordMaintenance()
286286
const auto conflictRecordPaths = _journal->conflictRecordPaths();
287287
for (const auto &path : conflictRecordPaths) {
288288
auto fsPath = _propagator->fullLocalPath(QString::fromUtf8(path));
289-
if (!QFileInfo(fsPath).exists()) {
289+
if (!QFileInfo::exists(fsPath)) {
290290
_journal->deleteConflictRecord(path);
291291
}
292292
}

0 commit comments

Comments
 (0)