Skip to content

Modernize ConfigCBDAccessSvc

Gerhard Raven requested to merge tweak-ConfigCDBAccessSvc into master
  • use dedicated type for read-only/read-write property
  • use string_views::to_string instead of equivalent free standing implementation
  • use boost::optional as return value instead of passing an uninitialized T by reference as INOUT argument, and returning a bool
  • call file() less often / less implicitly, and instead assign the return value to a variable, keeping it alive in the local scope (in preparation of (maybe) eventually adding a lock around it, so that the underlying file isn't closed while read operations are still 'in flight')
Edited by Marco Cattaneo

Merge request reports