FTK_RecTools: Do not put using declarations in the global namespace in headers.

Do not put using declarations in the global namespace in headers.
Also avoid copying a vector in argument passing.



Former-commit-id: 4118f09e

Trigger/TrigFTK/FTK_RecTools/FTK_RecTools/FTK_VertexFinderTool.h

@@ -44,7 +44,7 @@
 //#include "xAODTracking/TrackParticleContainerFwd.h"
 //#include "Tracking/TrkVertexFitter/TrkVxEdmCnv/TrkVxEdmCnv/IVxCandidateXAODVertex.h"
 class VxContainer;
-using std::vector;
+
 namespace Trk {
   class Track;
 //  class Trk::IVxCandidateXAODVertex;
@@ -115,11 +115,11 @@ class FTK_VertexFinderTool : public AthAlgTool, virtual public IFTK_VertexFinder
   //
   //  Helper functions with the uncerianties
   //
-	std::pair<xAOD::VertexContainer*, xAOD::VertexAuxContainer*> findVertex(vector<MyTrack> trks);
+    std::pair<xAOD::VertexContainer*, xAOD::VertexAuxContainer*> findVertex(std::vector<MyTrack>& trks);
   double ctheta2eta(double cot);
-  vector<MyTrack> getTracks(const FTK_RawTrackContainer* trks);
-  vector<MyTrack> getTracks(const TrackCollection* trks);
-  //tool handel
+  std::vector<MyTrack> getTracks(const FTK_RawTrackContainer* trks);
+  std::vector<MyTrack> getTracks(const TrackCollection* trks);
+  //tool handle
   ToolHandle<IFTK_UncertaintyTool> m_uncertaintyTool;
   ToolHandle<Trk::IVxCandidateXAODVertex> m_VertexEdmFactory;
 };

Trigger/TrigFTK/FTK_RecTools/src/FTK_VertexFinderTool.cxx

@@ -112,9 +112,8 @@ std::pair<xAOD::VertexContainer*, xAOD::VertexAuxContainer*> FTK_VertexFinderToo
 //
 // Covariance Matrix if there is a BLayer Hit
 //
-std::pair<xAOD::VertexContainer*, xAOD::VertexAuxContainer*> FTK_VertexFinderTool::findVertex(vector<MyTrack> mytrk)
+std::pair<xAOD::VertexContainer*, xAOD::VertexAuxContainer*> FTK_VertexFinderTool::findVertex(std::vector<MyTrack>& mytrk)
 {
-
   MsgStream athenaLog(msgSvc(), name());
   double mGkx=0.0001;
   double mGky=0.0001;