feature: New FairRun::AddTask(unique_ptr)

So that the callers can be more explicit about ownership,
add an alternative FairRun::AddTask that takes a
unique_ptr.

Do not yet deprecate the old API, because it's used in way
too many places. But we should update all our code to use
the new API so that the new style ist adopted.

examples/MQ/pixelDetector/macros/run_dAsciiSource.C

@@ -71,8 +71,8 @@ void run_dAsciiSource(TString mcEngine = "TGeant3")
     //  PixelDigiReadFromFile* digiRead = new PixelDigiReadFromFile();
     //  run->AddTask(digiRead);
 
-    PixelFindHits* hitFinderTask = new PixelFindHits();
-    run.AddTask(hitFinderTask);
+    auto hitFinderTask = std::make_unique<PixelFindHits>();
+    run.AddTask(std::move(hitFinderTask));
 
     run.Init();
 

examples/MQ/pixelDetector/macros/run_dBinSource.C

@@ -71,8 +71,8 @@ void run_dBinSource(TString mcEngine = "TGeant3")
     //  PixelDigiReadFromFile* digiRead = new PixelDigiReadFromFile();
     //  run->AddTask(digiRead);
 
-    PixelFindHits* hitFinderTask = new PixelFindHits();
-    run.AddTask(hitFinderTask);
+    auto hitFinderTask = std::make_unique<PixelFindHits>();
+    run.AddTask(std::move(hitFinderTask));
 
     run.Init();
 

fairroot/base/steer/FairRun.h

@@ -56,6 +56,7 @@ class FairRun : public TNamed
     /**
      * Add a FAIRTask to the simulation or analysis
      */
+    void AddTask(std::unique_ptr<FairTask> task) { AddTask(task.release()); }
     virtual void AddTask(FairTask* t);
     virtual void StoreTaskNames(const FairTask* t);
     virtual void SetTask(FairTask* t);