Skip to content

Commit 4ec3a76

Browse files
author
Michal Tichák
committed
perfect forwarding to reference
1 parent b9d26ef commit 4ec3a76

File tree

2 files changed

+6
-11
lines changed

2 files changed

+6
-11
lines changed

Utilities/Mergers/src/MergerAlgorithm.cxx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818

1919
#include "Mergers/MergeInterface.h"
2020
#include "Mergers/ObjectStore.h"
21+
#include "Framework/Logger.h"
2122

2223
#include <TEfficiency.h>
2324
#include <TGraph.h>
@@ -52,7 +53,7 @@ size_t estimateTreeSize(TTree* tree)
5253
// we need to recursively transverse whole TPad structure.
5354
auto collectUnderlyingObjects(TCanvas* canvas) -> std::vector<TObject*>
5455
{
55-
auto collectFromTPad = [](TPad* pad, std::vector<TObject*>& objects, auto&& collectFromTPad) {
56+
auto collectFromTPad = [](TPad* pad, std::vector<TObject*>& objects, const auto& collectFromTPad) {
5657
if (!pad) {
5758
return;
5859
}

Utilities/Mergers/test/test_Algorithm.cxx

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,6 @@
1414
///
1515
/// \author Piotr Konopka, piotr.jan.konopka@cern.ch
1616

17-
#include <boost/mpl/pair.hpp>
18-
#include <boost/test/tools/interface.hpp>
19-
#include <boost/test/tools/old/interface.hpp>
2017
#include <gsl/span>
2118
#include <memory>
2219
#include <stdexcept>
@@ -323,13 +320,10 @@ TCanvas* createCanvas(std::string name, std::string title, std::vector<std::shar
323320

324321
auto collectUnderlyingObjects(TCanvas* canvas) -> std::vector<TObject*>
325322
{
326-
if (!canvas) {
327-
return {};
328-
}
329-
330-
auto collectFromTPad = []<class TPadOrTCanvas>
331-
requires std::same_as<TPadOrTCanvas, TPad> || std::same_as<TPadOrTCanvas, TCanvas>
332-
(TPadOrTCanvas * pad, std::vector<TObject*> & objects, auto&& collectFromTPad) -> void {
323+
auto collectFromTPad = [](TPad* pad, std::vector<TObject*>& objects, const auto& collectFromTPad) {
324+
if (!pad) {
325+
return;
326+
}
333327
auto* primitives = pad->GetListOfPrimitives();
334328
for (int i = 0; i < primitives->GetSize(); ++i) {
335329
auto* primitive = primitives->At(i);

0 commit comments

Comments
 (0)