Skip to content
Snippets Groups Projects

Prefer CaloIndex enum over plain (unsigned) int, move some Calo Interfaces to Rec

Merged Gerhard Raven requested to merge prefer-enum-over-string into master

Prefer enum over plain (unsigned) int for increased type safety Move several Calo (Future) Interfaces into Rec, into the sole package using them.

must be applied in conjunction with Rec!1701 (merged), Phys!604 (merged), and Panoramix!53 (merged).

Edited by Marco Cattaneo

Merge request reports

Pipeline #1123493 passed

Pipeline passed for c396cd55 on prefer-enum-over-string

Approval is optional

Merged by Marco CattaneoMarco Cattaneo 5 years ago (Oct 3, 2019 7:48am UTC)

Merge details

  • Changes merged into master with 6c6e7f6e.
  • Deleted the source branch.

Pipeline #1129486 passed

Pipeline passed for 6c6e7f6e on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Gerhard Raven added 1 commit

    added 1 commit

    • b663f764 - Prefer enum over plain (unsigned) int for increased type safety

    Compare with previous version

  • Gerhard Raven added 1 commit

    added 1 commit

    • 91a6d266 - Prefer enum over plain (unsigned) int for increased type safety

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading