Skip to content

fix: push_back to emplace_back done by AI#5314

Merged
asalzburger merged 4 commits intoacts-project:mainfrom
asalzburger:fix-pushback-emplaceback
Apr 7, 2026
Merged

fix: push_back to emplace_back done by AI#5314
asalzburger merged 4 commits intoacts-project:mainfrom
asalzburger:fix-pushback-emplaceback

Conversation

@asalzburger
Copy link
Copy Markdown
Contributor

@asalzburger asalzburger commented Apr 3, 2026

PR Summary: Implicit conversion/narrowing + push_backemplace_back

*** This was done using Cursor on top of our SonarCloud report ***

What changed

push_backemplace_back

  • Replaced push_back with emplace_back in the Sonar-flagged call sites (and adjusted braced initialization where needed to keep the code compiling).
  • Updated pair/tuple insertions to use the correct emplace_back(...) argument form.
  • Converted tapVec.push_back(std::make_pair(...)) to tapVec.emplace_back(..., ...) in CuboidVolumeBuilder.cpp.

--- END COMMIT MESSAGE ---

Any further description goes here, @-mentions are ok here!

  • Use a conventional commits prefix: quick summary
    • We mostly use feat, fix, refactor, docs, chore and build types.
  • A milestone will be assigned by one of the maintainers

@github-actions github-actions bot added Component - Core Affects the Core module Event Data Model labels Apr 3, 2026
@github-actions github-actions bot added this to the next milestone Apr 3, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 3, 2026

📊: Physics performance monitoring for d5df008

Full contents

physmon summary

Copy link
Copy Markdown
Contributor Author

@asalzburger asalzburger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adressing PR comments - by passing arguments directly.

@asalzburger asalzburger requested a review from andiwand April 4, 2026 05:02
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 7, 2026

@asalzburger asalzburger merged commit f1f3072 into acts-project:main Apr 7, 2026
43 checks passed
@andiwand andiwand modified the milestones: next, v46.2.0 Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants