Skip to content

Commit 1da3c10

Browse files
committed
[VISUALIZATION] Apply code checks/format
1 parent b205513 commit 1da3c10

File tree

3 files changed

+14
-11
lines changed

3 files changed

+14
-11
lines changed

Fireworks/Core/src/FWEveViewManager.cc

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
// system include files
1414

1515
#include <functional>
16+
#include <memory>
1617

1718
// user include files
1819
#include "TEveManager.h"
@@ -278,25 +279,25 @@ FWViewBase* FWEveViewManager::buildView(TEveWindowSlot* iParent, const std::stri
278279
std::shared_ptr<FWEveView> view;
279280
switch (type) {
280281
case FWViewType::k3D:
281-
view.reset(new FW3DView(iParent, type));
282+
view = std::make_shared<FW3DView>(iParent, type);
282283
break;
283284
case FWViewType::kISpy:
284-
view.reset(new FWISpyView(iParent, type));
285+
view = std::make_shared<FWISpyView>(iParent, type);
285286
break;
286287
case FWViewType::kRhoPhi:
287288
case FWViewType::kRhoZ:
288289
case FWViewType::kRhoPhiPF:
289-
view.reset(new FWRPZView(iParent, type));
290+
view = std::make_shared<FWRPZView>(iParent, type);
290291
break;
291292
case FWViewType::kLego:
292293
case FWViewType::kLegoPFECAL:
293-
view.reset(new FWEveLegoView(iParent, type));
294+
view = std::make_shared<FWEveLegoView>(iParent, type);
294295
break;
295296
case FWViewType::kLegoHF:
296-
view.reset(new FWHFView(iParent, type));
297+
view = std::make_shared<FWHFView>(iParent, type);
297298
break;
298299
case FWViewType::kGlimpse:
299-
view.reset(new FWGlimpseView(iParent, type));
300+
view = std::make_shared<FWGlimpseView>(iParent, type);
300301
break;
301302
default:
302303
break;

Fireworks/Core/src/FWGeometryTableViewManager.cc

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
//
1212

1313
#include <functional>
14+
#include <memory>
1415

1516
#include "TFile.h"
1617
#include "TSystem.h"
@@ -48,9 +49,9 @@ FWViewBase* FWGeometryTableViewManager::buildView(TEveWindowSlot* iParent, const
4849
FWViewType::EType typeId =
4950
(type == FWViewType::sName[FWViewType::kGeometryTable]) ? FWViewType::kGeometryTable : FWViewType::kOverlapTable;
5051
if (typeId == FWViewType::kGeometryTable)
51-
view.reset(new FWGeometryTableView(iParent, &colorManager()));
52+
view = std::make_shared<FWGeometryTableView>(iParent, &colorManager());
5253
else
53-
view.reset(new FWOverlapTableView(iParent, &colorManager()));
54+
view = std::make_shared<FWOverlapTableView>(iParent, &colorManager());
5455

5556
view->setBackgroundColor();
5657
m_views.push_back(std::shared_ptr<FWGeometryTableViewBase>(view));

Fireworks/Core/src/FWTriggerTableViewManager.cc

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,9 @@
1212

1313
// system include files
1414
#include <cassert>
15-
#include <iostream>
1615
#include <functional>
16+
#include <iostream>
17+
#include <memory>
1718

1819
// user include files
1920

@@ -39,9 +40,9 @@ class FWViewBase* FWTriggerTableViewManager::buildView(TEveWindowSlot* iParent,
3940
std::shared_ptr<FWTriggerTableView> view;
4041

4142
if (type == FWViewType::sName[FWViewType::kTableHLT])
42-
view.reset(new FWHLTTriggerTableView(iParent));
43+
view = std::make_shared<FWHLTTriggerTableView>(iParent);
4344
else
44-
view.reset(new FWL1TriggerTableView(iParent));
45+
view = std::make_shared<FWL1TriggerTableView>(iParent);
4546

4647
view->setProcessList(&(context().metadataManager()->processNamesInJob()));
4748

0 commit comments

Comments
 (0)