Skip to content

Commit c18bd3c

Browse files
committed
refactor(discovery): ranges, ref captures, precision, const localJob
Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
1 parent d3cf9f5 commit c18bd3c

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

src/libsync/discovery.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2113,7 +2113,7 @@ QStringList ProcessDirectoryJob::queryEditorsKeepingFileBusy(const SyncFileItemP
21132113
}
21142114

21152115
if (const auto isMatchingFileExtension = std::ranges::find_if(kFileExtsForSigning,
2116-
[path](const auto &matchingExtension) {
2116+
[&path](const auto &matchingExtension) {
21172117
return path._local.endsWith(matchingExtension, Qt::CaseInsensitive);
21182118
}) != std::cend(kFileExtsForSigning);
21192119
!isMatchingFileExtension) {
@@ -2124,8 +2124,8 @@ QStringList ProcessDirectoryJob::queryEditorsKeepingFileBusy(const SyncFileItemP
21242124
const auto editorsKeepingFileBusy = Utility::queryProcessInfosKeepingFileOpen(fullLocalPath);
21252125

21262126
for (const auto &detectedEditorName : editorsKeepingFileBusy) {
2127-
const auto isMatchingEditorFound = std::find_if(std::cbegin(editorNamesForDelayedUpload), std::cend(editorNamesForDelayedUpload),
2128-
[detectedEditorName](const auto &matchingEditorName) {
2127+
const auto isMatchingEditorFound = std::ranges::find_if(editorNamesForDelayedUpload,
2128+
[&detectedEditorName](const auto &matchingEditorName) {
21292129
return detectedEditorName.processName.startsWith(matchingEditorName, Qt::CaseInsensitive);
21302130
}) != std::cend(editorNamesForDelayedUpload);
21312131
if (isMatchingEditorFound) {
@@ -2185,7 +2185,7 @@ void ProcessDirectoryJob::subJobFinished()
21852185
if (job->_dirItem)
21862186
emit _discoveryData->itemDiscovered(job->_dirItem);
21872187

2188-
int count = _runningJobs.removeAll(job);
2188+
const auto count = _runningJobs.removeAll(job);
21892189
ASSERT(count == 1);
21902190
job->deleteLater();
21912191
QTimer::singleShot(0, _discoveryData, &DiscoveryPhase::scheduleMoreJobs);
@@ -2366,7 +2366,7 @@ void ProcessDirectoryJob::setFolderQuota(const FolderQuota &folderQuota)
23662366
void ProcessDirectoryJob::startAsyncLocalQuery()
23672367
{
23682368
QString localPath = _discoveryData->_localDir + _currentFolder._local;
2369-
auto localJob = new DiscoverySingleLocalDirectoryJob(_discoveryData->_account, localPath, _discoveryData->_syncOptions._vfs.data(), _discoveryData->_fileSystemReliablePermissions);
2369+
auto *const localJob = new DiscoverySingleLocalDirectoryJob(_discoveryData->_account, localPath, _discoveryData->_syncOptions._vfs.data(), _discoveryData->_fileSystemReliablePermissions);
23702370

23712371
_discoveryData->_currentlyActiveJobs++;
23722372
_pendingAsyncJobs++;

0 commit comments

Comments
 (0)