Skip to content

NSW Trigger Simulation - MicroMegas: smart pointers follow-up

This is a follow-up MR, in order to address several specific comments of the previous one !45672 (merged), concerning the migration from normal pointers to smart ones. Going into details:

  • All standard pointers were moved to smart ones: shared_ptr or unique_ptr
  • Some classes don't have a copy of pointers among private members anymore: those pointers are passed by reference now
  • The added extra functions for initialization and deletion of such pointers are removed, since not needed anymore
  • Added a few very minor changes which don't alter the general structure of the code/packets, but just fix some inconsistencies of the previous MR: they're just one new line, one line added to handle a specific case, a small change to reflect C++ syntax and a change about range-based for-loops (the last two points were raised among the final comments of the previous MR). I totally know that's not the correct practice but, since they're not absolutely as important as the aforementioned three points, I seized the opportunity to add them here... Please, forgive me!

./cc @iodice @rosati @vadamico @gkehris @sfrances

Edited by Francesco Giuseppe Gravili

Merge request reports