Skip to content
Snippets Groups Projects

[PrSelection] spell out templated typenames in exceptions

Merged Paul Seyfert requested to merge pseyfert/Rec:mod_olli into olupton_pr_filter_container
All threads resolved!

Rather than putting bare <T> into the gaudi exception explanation, we can use the compile-time known type name. It might not resolve all questions one might face when seeing the exception, but at least disambiguates if incompatible track, particle, younameit selections are overflowing/setoperated.

Merge request reports

Pipeline #598169 failed

Pipeline failed for 7532d39c on pseyfert:mod_olli

Merged by Olli LuptonOlli Lupton 6 years ago (Nov 21, 2018 12:50pm UTC)

Loading

Pipeline #598181 passed

Pipeline passed for f67dc122 on olupton_pr_filter_container

Activity

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

    added 1 commit

    • 7532d39c - Define details::typename_v helper following @graven's suggestion.

    Compare with previous version

  • Olli Lupton resolved all discussions

    resolved all discussions

  • Olli Lupton approved this merge request

    approved this merge request

  • Olli Lupton mentioned in commit f67dc122

    mentioned in commit f67dc122

  • merged

  • Olli Lupton mentioned in merge request !1281 (merged)

    mentioned in merge request !1281 (merged)

  • Please register or sign in to reply
    Loading