Skip to content

Commit

Permalink
Simplify size checks
Browse files Browse the repository at this point in the history
  • Loading branch information
jsamudio committed Apr 10, 2024
1 parent cee5119 commit cfe5347
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -54,12 +54,9 @@ void LegacyPFRecHitProducer::produce(edm::Event& event, const edm::EventSetup& s
const reco::PFRecHitHostCollection::ConstView& alpakaPfRecHits = pfRecHitsAlpakaSoA.const_view();

reco::PFRecHitCollection out;
int reserveSize = 0;
if (alpakaPfRecHits.metadata().size() != 0)
reserveSize = alpakaPfRecHits.size();
out.reserve(reserveSize);

if (alpakaPfRecHits.metadata().size() != 0) {
out.reserve(alpakaPfRecHits.size());
for (size_t i = 0; i < alpakaPfRecHits.size(); i++) {
reco::PFRecHit& pfrh =
out.emplace_back(caloGeo_.at(alpakaPfRecHits[i].layer())->getGeometry(alpakaPfRecHits[i].detId()),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ namespace ALPAKA_ACCELERATOR_NAMESPACE {

reco::PFRecHitDeviceCollection pfRecHits{(int)num_recHits, event.queue()};

if (pfRecHits->metadata().size() != 0) {
if (num_recHits != 0) {
PFRecHitProducerKernel<CAL> kernel{event.queue(), num_recHits};
for (const auto& token : recHitsToken_)
kernel.processRecHits(
Expand Down

0 comments on commit cfe5347

Please sign in to comment.