KalmanVertexUpdator performance tuning for use in AMVF

This MR significantly speeds up the KalmanVertexUpdator's update method by replacing a temporarily created and subsequently destroyed xAOD::Vertex object by a slim struct. This is also done in the IVertexUpdator and propagated to all clients. This, together with local caching of a matrix inversion result, yields a factor of up to 4 speed improvement for the Adaptive multi vertex fitter (see https://its.cern.ch/jira/browse/ATLIDTRKCP-245).

In addition to this performance tuning, some clean-up to the code is performed (removal of commented lines, attempt to clean up indentation to a consistent scheme, uniform use of messaging macros, use of std algorithms where possible, fix of potential memory leak).

The changes were tested to not change the reco output (except for CPU/timings) compared to running the AMVF in an unchanged master.

Adding @sroe and @npetters who are also working on improving the AMVF performance.

Tracking/TrkVertexFitter/TrkVertexFitterUtils/src/KalmanVertexUpdator.cxx

@@ -68,24 +68,9 @@ namespace Trk{
    }
 
    //trying to find requested track
-   std::vector<Trk::VxTrackAtVertex>::iterator lintracksBegin  = tracksAtVertex->begin();
-   std::vector<Trk::VxTrackAtVertex>::iterator lintracksEnd    = tracksAtVertex->end();
+   auto righttrack = std::find(tracksAtVertex->begin(), tracksAtVertex->end(), trk); 
 
-   std::vector<Trk::VxTrackAtVertex>::iterator righttrack = lintracksBegin;
-
-   //to be replaced with faster/smarter search
-   bool found(false);
-   for (std::vector<Trk::VxTrackAtVertex>::iterator iter=lintracksBegin;iter!=lintracksEnd;++iter)
-   {
-     if (trk == *iter)
-     {
-       found = true;
-       righttrack = iter;
-       break;
-     }
-   }//end for loop
-
-   if (!found)
+   if (righttrack == tracksAtVertex->end())
    {
      ATH_MSG_VERBOSE ("The track requested for removal or adding is not found in the vector of tracks");
      if (sign<0) {