diff --git a/PhysicsAnalysis/AnalysisEventDisplay/xAODJiveXML/src/xAODJetRetriever.cxx b/PhysicsAnalysis/AnalysisEventDisplay/xAODJiveXML/src/xAODJetRetriever.cxx
index e2cfe665a2b87e1d2990faab2c0c727e71f30c2e..60fbf5d24c1267658c0db86a0fd77948dee4cd8e 100755
--- a/PhysicsAnalysis/AnalysisEventDisplay/xAODJiveXML/src/xAODJetRetriever.cxx
+++ b/PhysicsAnalysis/AnalysisEventDisplay/xAODJiveXML/src/xAODJetRetriever.cxx
@@ -86,7 +86,7 @@ namespace JiveXML {
    * Retrieve basic parameters, mainly four-vectors, for each collection.
    * Also association with clusters and tracks (ElementLink).
    */
-  const DataMap xAODJetRetriever::getData(const xAOD::JetContainer* jetCont, std::string jetkey) {
+  const DataMap xAODJetRetriever::getData(const xAOD::JetContainer* jetCont, const std::string &jetkey) {
 
     ATH_MSG_DEBUG( "in getData()" );
 
diff --git a/PhysicsAnalysis/AnalysisEventDisplay/xAODJiveXML/xAODJiveXML/xAODJetRetriever.h b/PhysicsAnalysis/AnalysisEventDisplay/xAODJiveXML/xAODJiveXML/xAODJetRetriever.h
index d5600eeedcf33abdc01e15d69f6e0f17d0356033..99f4b55d7db34a390150f5371c9ff48d59271d26 100755
--- a/PhysicsAnalysis/AnalysisEventDisplay/xAODJiveXML/xAODJiveXML/xAODJetRetriever.h
+++ b/PhysicsAnalysis/AnalysisEventDisplay/xAODJiveXML/xAODJiveXML/xAODJetRetriever.h
@@ -47,7 +47,7 @@ namespace JiveXML{
 
     virtual StatusCode retrieve(ToolHandle<IFormatTool> &FormatTool);
     /// Puts the variables into a DataMap
-    const DataMap getData(const xAOD::JetContainer*, std::string jetkey);
+    const DataMap getData(const xAOD::JetContainer*, const std::string &jetkey);
     /// Gets the StoreGate keys for the desired containers
     const std::vector<std::string> getKeys();
     /// Return the name of the data type that is generated by this retriever
diff --git a/Tracking/TrkEventCnv/TrkJiveXML/TrkJiveXML/TrackRetriever.h b/Tracking/TrkEventCnv/TrkJiveXML/TrkJiveXML/TrackRetriever.h
index a7c80abf2cf8b92dacbcb032f13e077d34fb3d1d..3d93201e0b681816283467cc5871a5e2c0bb202e 100644
--- a/Tracking/TrkEventCnv/TrkJiveXML/TrkJiveXML/TrackRetriever.h
+++ b/Tracking/TrkEventCnv/TrkJiveXML/TrkJiveXML/TrackRetriever.h
@@ -70,7 +70,7 @@ namespace JiveXML{
 
     virtual StatusCode retrieve(ToolHandle<IFormatTool> &FormatTool);
     /// Puts the variables into a DataMap
-    const DataMap getData(const TrackCollection* trackCollection, const std::string collectionName);
+    const DataMap getData(const TrackCollection* trackCollection, const std::string &collectionName);
     /// Gets the StoreGate keys for the desired containers
     const std::vector<std::string> getKeys();
     /// Return the name of the data type that is generated by this retriever
@@ -96,7 +96,7 @@ namespace JiveXML{
     ToolHandle< Trk::ITrackSummaryTool > m_trackSumTool{this, "TrackSummaryTool", "Trk::TrackSummaryTool/InDetTrackSummaryTool", "Track Summary Tool for number of Pix/SCT/TRT hits"};
 
     /// Used to find out the corresponding sub-det from ROT->identify().
-    const AtlasDetectorID* m_idHelper;
+    const AtlasDetectorID* m_idHelper{};
 
   };
 
diff --git a/Tracking/TrkEventCnv/TrkJiveXML/src/TrackRetriever.cxx b/Tracking/TrkEventCnv/TrkJiveXML/src/TrackRetriever.cxx
index e3c6fde3c542f319a1112bf872475841c06ae4b2..6716b4ac999aac5a14269f1d3bd9155b137e1cee 100644
--- a/Tracking/TrkEventCnv/TrkJiveXML/src/TrackRetriever.cxx
+++ b/Tracking/TrkEventCnv/TrkJiveXML/src/TrackRetriever.cxx
@@ -390,7 +390,7 @@ namespace JiveXML {
     return StatusCode::SUCCESS;
   }
 
-  const DataMap TrackRetriever::getData(const TrackCollection* trackCollection, const std::string collectionName) {
+  const DataMap TrackRetriever::getData(const TrackCollection* trackCollection, const std::string &collectionName) {
 
     ATH_MSG_DEBUG("in getData()");