Don't bother creating a unique ptr for computeChecksum as we never use the guarded ptr

diff --git a/src/libsync/bulkpropagatorjob.cpp b/src/libsync/bulkpropagatorjob.cpp
index dab9f984687bec40a97af5cd76964a21a543362b..450d09e9c710b5dd67189775a359cc4e34a12ac7 100644 (file)
--- a/src/libsync/bulkpropagatorjob.cpp
+++ b/src/libsync/bulkpropagatorjob.cpp
@@ -260,20 +260,16 @@ void BulkPropagatorJob::slotComputeTransmissionChecksum(SyncFileItemPtr item,
                                                         UploadFileInfo fileToUpload)
 {
     // Compute the transmission checksum.
-    auto computeChecksum = std::make_unique<ComputeChecksum>(this);
-    if (uploadChecksumEnabled()) {
-        computeChecksum->setChecksumType("MD5" /*propagator()->account()->capabilities().uploadChecksumType()*/);
-    } else {
-        computeChecksum->setChecksumType(QByteArray());
-    }
+    const auto computeChecksum = new ComputeChecksum(this);
+    const auto checksumType = uploadChecksumEnabled() ? "MD5" : "";
+    computeChecksum->setChecksumType(checksumType);
 
-    connect(computeChecksum.get(), &ComputeChecksum::done,
-            this, [this, item, fileToUpload] (const QByteArray &contentChecksumType, const QByteArray &contentChecksum) {
+    connect(computeChecksum, &ComputeChecksum::done, this, [this, item, fileToUpload] (const QByteArray &contentChecksumType, const QByteArray &contentChecksum) {
         slotStartUpload(item, fileToUpload, contentChecksumType, contentChecksum);
     });
-    connect(computeChecksum.get(), &ComputeChecksum::done,
-            computeChecksum.get(), &QObject::deleteLater);
-    computeChecksum.release()->start(fileToUpload._path);
+    connect(computeChecksum, &ComputeChecksum::done, computeChecksum, &QObject::deleteLater);
+
+    computeChecksum->start(fileToUpload._path);
 }
 
 void BulkPropagatorJob::slotStartUpload(SyncFileItemPtr item,