Skip to content

Commit

Permalink
Merge pull request #47227 from makortel/alpakaProducerRemoveDefaultCo…
Browse files Browse the repository at this point in the history
…nstructor

Remove default constructor from Alpaka EDProducer base classes
  • Loading branch information
cmsbuild authored Feb 4, 2025
2 parents 753c9f9 + b7ad19d commit e983e10
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 5 deletions.
2 changes: 0 additions & 2 deletions HeterogeneousCore/AlpakaCore/interface/alpaka/ProducerBase.h
Original file line number Diff line number Diff line change
Expand Up @@ -46,8 +46,6 @@ namespace ALPAKA_ACCELERATOR_NAMESPACE {
using Base = BaseT<Args..., edm::Transformer>;

public:
// TODO: default constructor to be removed after all derived classes have been migrated
ProducerBase() : backendToken_(Base::produces("backend")) {}
ProducerBase(edm::ParameterSet const& iConfig)
: backendToken_(Base::produces("backend")),
// The 'synchronize' parameter can be unset in Alpaka
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ namespace ALPAKA_ACCELERATOR_NAMESPACE {
using Base = ProducerBase<edm::global::EDProducer, Args...>;

protected:
EDProducer() = default; // to be removed in the near future
EDProducer(edm::ParameterSet const iConfig) : Base(iConfig) {}

public:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ namespace ALPAKA_ACCELERATOR_NAMESPACE {
using Base = ProducerBase<edm::stream::EDProducer, Args...>;

protected:
EDProducer() = default; // to be removed in the near future
EDProducer(edm::ParameterSet const iConfig) : Base(iConfig) {}

public:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ namespace ALPAKA_ACCELERATOR_NAMESPACE {
using Base = ProducerBase<edm::stream::EDProducer, edm::ExternalWork, Args...>;

protected:
SynchronizingEDProducer() = default; // to be removed in the near future
SynchronizingEDProducer(edm::ParameterSet const iConfig) : Base(iConfig) {}

public:
Expand Down

0 comments on commit e983e10

Please sign in to comment.