Skip to content

enable ADL lookup for binary Time,Time and TimeSpan,TimeSpan operators

Gerhard Raven requested to merge lhcb/Gaudi:allow_adl_in_time into master

This merge request allows the removal of namespace Gaudi { using operator::<; } which appears in eg. [CondDBTimeSwitchSvc] (https://gitlab.cern.ch/lhcb/LHCb/blob/b6097055a432ded300308823f87ccab7bc72d688/Det/DetCond/src/component/CondDBTimeSwitchSvc.cpp#L29).

In addition, do some drive-by modernization:

  • remove 'void' as argument specification on functions
  • prefer struct for classes with only public members/functions
  • prefer compiler generated destructors
  • prefer = default over explicit implementations that do the same

Merge request reports

Loading