Skip to content

Modernize TrackTools

Gerhard Raven requested to merge modernize-tracktools into master
  • prefer inheriting constructors
  • prefer implicit destructors
  • prefer Gaudi::Property
  • prefer direct member initialization
  • prefer standalone functions in anonymous namespace over member functions that do not access any class data and/or methods
  • prefer STL algorithms over raw loops
  • prefer std::unique_ptr over raw pointers + explicit delete
  • prefer switch over if-then-elseif-elseif-else
  • prefer range-based for loops
  • move loop invariants out of loop body
  • remove obsolete SVN/CVS keywords
  • prefer auto
  • add final
  • prefer emplace_back
  • prefer inheriting from extends over virtual inheritance
  • prefer std::initializer_list over boost::assign
  • prefer std::unique_ptr and move semantics over std::shared_ptr and copy
  • prefer std::make_unique
  • prefer range-based for loops over BOOST_FOREACH
  • do not cache msgLevel(MSG::DEBUG)

Merge request reports