Skip to content
Snippets Groups Projects

ToolHandle: add EnableTool tagged bool to control retrieval

Merged Charles Leggett requested to merge leggett/Gaudi:dev/ToolHandleTaggedBoolUse into master

The syntax of

  ToolHandle<IMyTool> m_tool;
  bool m_useToolFlag;
...
  ATH_CHECK( m_tool.retrieve( DisableTool{ !m_useToolFlag } ));

is a little confusing. This MR adds another TaggedBool to make it a little clearer:

   ATH_CHECK( m_tool.retrieve( EnableTool{ m_useToolFlag } ));

what are people's thoughts on DisableTool / EnableTool vs. DisableToolIf / EnableToolIf

Edited by Charles Leggett

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
Please register or sign in to reply
Loading