# HG changeset patch
# User Alain Mazy <am@orthanc.team>
# Date 1762796659 -3600
# Node ID 0b94f843be89683d894d2d546c5f04dbcedcd244
# Parent  dfb2085b778c575972294245ddc419f1b7fcacff
cppcheck: invalidLifetime -> re-enabled JobsEngine.SequenceOfOperationsJob

diff -r dfb2085b778c -r 0b94f843be89 OrthancFramework/UnitTestsSources/JobsTests.cpp
--- a/OrthancFramework/UnitTestsSources/JobsTests.cpp	Mon Nov 10 18:39:19 2025 +0100
+++ b/OrthancFramework/UnitTestsSources/JobsTests.cpp	Mon Nov 10 18:44:19 2025 +0100
@@ -733,25 +733,26 @@
 }
 
 
-TEST(JobsEngine, DISABLED_SequenceOfOperationsJob)
+TEST(JobsEngine, SequenceOfOperationsJob)
 {
   JobsEngine engine(10);
   engine.SetThreadSleep(10);
   engine.SetWorkersCount(3);
   engine.Start();
 
-  SequenceOfOperationsJob* job = NULL;
-
   {
     std::string id;
-    std::unique_ptr<SequenceOfOperationsJob> a(new SequenceOfOperationsJob);
-    job = a.get();
-    engine.GetRegistry().Submit(id, a.release(), 0);
+    std::unique_ptr<SequenceOfOperationsJob> seq(new SequenceOfOperationsJob);
+    engine.GetRegistry().Submit(id, seq.release(), 0);
   }
 
   boost::this_thread::sleep(boost::posix_time::milliseconds(500));
 
   {
+    SequenceOfOperationsJob* job = NULL;
+    std::unique_ptr<SequenceOfOperationsJob> seq(new SequenceOfOperationsJob);
+    job = seq.get();
+
     SequenceOfOperationsJob::Lock lock(*job);
     size_t i = lock.AddOperation(new LogJobOperation);
     size_t j = lock.AddOperation(new LogJobOperation);
diff -r dfb2085b778c -r 0b94f843be89 OrthancServer/Sources/ServerJobs/StorageCommitmentScpJob.cpp
--- a/OrthancServer/Sources/ServerJobs/StorageCommitmentScpJob.cpp	Mon Nov 10 18:39:19 2025 +0100
+++ b/OrthancServer/Sources/ServerJobs/StorageCommitmentScpJob.cpp	Mon Nov 10 18:44:19 2025 +0100
@@ -420,7 +420,6 @@
                                                    const Json::Value& serialized) :
     SetOfCommandsJob(new Unserializer(*this), serialized),
     context_(context),
-    ready_(false),
     transactionUid_(SerializationToolbox::ReadString(serialized, TRANSACTION_UID)),
     calledAet_(SerializationToolbox::ReadString(serialized, CALLED_AET))
     // "ready_" is initialized by the unserializer