Eagle plugin: Added 'static' and 'const T&' as applicable

common/eagle_parser.cpp

@@ -195,7 +195,7 @@ EJUNCTION::EJUNCTION( wxXmlNode* aJunction)
     y    = parseRequiredAttribute<double>( aJunction, "y" );
 }
 
-ELABEL::ELABEL( wxXmlNode* aLabel, wxString aNetName)
+ELABEL::ELABEL( wxXmlNode* aLabel, const wxString& aNetName )
 {
     /*
     <!ELEMENT label EMPTY>

eeschema/sch_eagle_plugin.cpp

@@ -50,7 +50,7 @@ constexpr double EUNIT_TO_MIL = 1000.0 / 25.4;
 
 using namespace std;
 
-NODE_MAP mapChildren( wxXmlNode* aCurrentNode )
+static NODE_MAP mapChildren( wxXmlNode* aCurrentNode )
 {
     // Map node_name -> node_pointer
     NODE_MAP nodesMap;
@@ -71,7 +71,7 @@ NODE_MAP mapChildren( wxXmlNode* aCurrentNode )
     return nodesMap;
 }
 
-int countChildren( wxXmlNode* aCurrentNode, std::string name)
+static int countChildren( wxXmlNode* aCurrentNode, const std::string& aName )
 {
     // Map node_name -> node_pointer
     int count = 0;

eeschema/sch_eagle_plugin.h

@@ -117,9 +117,10 @@ private:
     void loadModuleinst( wxXmlNode* aModuleinstNode );
     void loadLibrary( wxXmlNode* aLibraryNode );
 
-    void                loadSegments( wxXmlNode* aSegmentsNode, wxString aNetName, wxString aNetClass );
+    void                loadSegments( wxXmlNode* aSegmentsNode, const wxString& aNetName,
+            const wxString& aNetClass );
     SCH_LINE*           loadSignalWire( wxXmlNode* aWireNode );
-    SCH_GLOBALLABEL*    loadLabel( wxXmlNode* aLabelNode, wxString aNetName );
+    SCH_GLOBALLABEL*    loadLabel( wxXmlNode* aLabelNode, const wxString& aNetName );
     SCH_JUNCTION*       loadJunction( wxXmlNode* aJunction );
 
     LIB_PART*       loadSymbol( wxXmlNode* aSymbolNode );

include/eagle_parser.h

@@ -448,7 +448,7 @@ struct ELABEL
     wxString netname;
 
 
-    ELABEL( wxXmlNode* aLabel, wxString aNetName );
+    ELABEL( wxXmlNode* aLabel, const wxString& aNetName );
 };