Skip to content
Snippets Groups Projects

migrate G4EmProcessOptions.hh to G4EmParameters.hh

All threads resolved!

Migration to use directly G4EmParameters instead of G4EmProcessOptions as advised by Geant4 people.

It is potentially a breaking change because it merges two properties of hysicsListToolBase into one. This was advised decided upon discussion in ATLASSIM-3967.

With a LXR search only one client using these properties was found and the settings were commented out. This MR adds an additional comment as a warning.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Miha Muskinja resolved all discussions

    resolved all discussions

  • added review-approved label and removed review-pending-level-1 label

  • merged

  • Walter Lampl mentioned in commit b6440c58

    mentioned in commit b6440c58

  • Please register or sign in to reply
    Loading