Remove some memory leaks

andresailer commented 5 years ago

This fixes a number of memory leaks we observe in our tests.

BEGINRELEASENOTES

LCIOStorer: fix memory leak of PIDHandler
LCFIPlusProcessor: clean algos and params after the end() of algos, fix memory leaks
Process: fix memory leak of double *ymin, use std::vector<double>
Process: use shared_ptr for JetFinder, fixes memory leak
VertexFitterSimple: fix memory leak for IP: only create it if it is used
VertexFinderSuehara: fix mismatch between new[] and delete (w/o []), use std::vector

ENDRELEASENOTES

jstrube commented 5 years ago

Looks good to me. Thanks for submitting these fixes.

andresailer commented 5 years ago

Are there any objections for these changes? Can it be merged?

lcfiplus / LCFIPlus