Skip to content

cmExportSet: replace manual memory deallocation with std::unique_ptr

Tushar Maheshwari requested to merge tusharpm/cmake:smart_ptr/cmExportSet into master
  • Remove cmDeleteAll calls from the destructors of cmExportSet and cmExportSetMap.
  • Optimize ownership semantics for cmExportSetMap.

Questions:

  • Should the std::unique_ptr<...> const& be replaced with auto const& outside the class definitions?
  • Is cmExportSetMap trivial enough to be moved into cmExportSet.{cxx,h}?

Merge request reports