Don't bother creating a unique ptr for PutMultiFileJob as we never use the guarded ptr
authorClaudio Cambra <claudio.cambra@nextcloud.com>
Thu, 11 May 2023 11:47:55 +0000 (19:47 +0800)
committerClaudio Cambra <claudio.cambra@nextcloud.com>
Mon, 15 May 2023 12:46:21 +0000 (20:46 +0800)
Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
src/libsync/bulkpropagatorjob.cpp

index f414c73f3b21488c3391e96f3137d3da38744634..bbbc98ab1b6fb39ed5955fb79c9c135eb97e7b5b 100644 (file)
@@ -219,19 +219,18 @@ void BulkPropagatorJob::triggerUpload()
     }
 
     const auto bulkUploadUrl = Utility::concatUrlPath(propagator()->account()->url(), QStringLiteral("/remote.php/dav/bulk"));
-    auto job = std::make_unique<PutMultiFileJob>(propagator()->account(), bulkUploadUrl, std::move(uploadParametersData), this);
-    connect(job.get(), &PutMultiFileJob::finishedSignal, this, &BulkPropagatorJob::slotPutFinished);
+    auto job = new PutMultiFileJob(propagator()->account(), bulkUploadUrl, std::move(uploadParametersData), this);
+    connect(job, &PutMultiFileJob::finishedSignal, this, &BulkPropagatorJob::slotPutFinished);
 
     for(auto &singleFile : _filesToUpload) {
-        connect(job.get(), &PutMultiFileJob::uploadProgress,
-                this, [this, singleFile] (qint64 sent, qint64 total) {
+        connect(job, &PutMultiFileJob::uploadProgress, this, [this, singleFile] (const qint64 sent, const qint64 total) {
             slotUploadProgress(singleFile._item, sent, total);
         });
     }
 
-    adjustLastJobTimeout(job.get(), timeout);
-    _jobs.append(job.get());
-    job.release()->start();
+    adjustLastJobTimeout(job, timeout);
+    _jobs.append(job);
+    job->start();
     if (parallelism() == PropagatorJob::JobParallelism::FullParallelism && _jobs.size() < parallelJobsMaximumCount) {
         scheduleSelfOrChild();
     }