Skip to content

Replace DataObjID{,Coll}Property with Gaudi::Property<DataObjID{,Coll}>

Gerhard Raven requested to merge graven/Gaudi:simplify-DataObjIDProperty into master

Fixes #1 (closed)

  • re-order includes so that DataObjID.h is included prior to Property.h
  • remove include of Property.h from TypeNameString.h, and add an alternative to the TypeNameString constructor from Gaudi::Property (without mentioning Gaudi::Property) to avoid a very early inclusion of Property.h, which makes it very difficult to add a parse function to the overload set in Gaudi::Parsers prior to its usage in Property.h
  • add parse and toStream as friend functions to DataObjID (required for Gaudi::Property<DataObjID>)
  • add a grammar for parsing DataObjID which can be used by the parser of DataObjIDColl = std::unordered_set<DataObjID> as 'inner' parser
  • add Gaudi::Parsers::parse overload for DataObjIDColl (required for Gaudi::Property<DataObjIDColl>)
  • remove dedicated DataObjID{,Coll,Property class, but provide a backwards compatible 'using' alias for them -- and mark their use as deprecated
  • add (outer) parsers for std::unordered_{map,set} to GrammarsV2.h
  • add toStream for std::unordered_{map,set} to ToStream.h
Edited by Gerhard Raven

Merge request reports