Linter fixes

src/Mod/Material/App/FolderTree.h

@@ -56,7 +56,7 @@ public:
         _type = type;
     }
 
-    const std::shared_ptr<std::map<QString, std::shared_ptr<FolderTreeNode<T>>>> getFolder() const
+    std::shared_ptr<std::map<QString, std::shared_ptr<FolderTreeNode<T>>>> getFolder() const
     {
         assert(_type == NodeType::FolderNode);
         return _folder;
@@ -87,7 +87,7 @@ public:
         setType(NodeType::DataNode);
         _data = data;
     }
-    void setUUID(const QString uuid)
+    void setUUID(const QString& uuid)
     {
         setType(NodeType::DataNode);
         _uuid = uuid;

src/Mod/Material/App/Library.cpp

@@ -97,8 +97,6 @@ bool Library::isRoot(const QString& path) const
 {
     QString localPath = getLocalPath(path);
     QString cleanPath = getLocalPath(QStringLiteral(""));
-    std::string pLocal = localPath.toStdString();
-    std::string pclean = cleanPath.toStdString();
     return (cleanPath == localPath);
 }
 

tests/src/Mod/Material/App/TestMaterials.cpp

@@ -31,8 +31,8 @@
 #include <QString>
 
 #include <App/Application.h>
-#include <Base/Quantity.h>
 #include <Base/Interpreter.h>
+#include <Base/Quantity.h>
 #include <Gui/MetaTypes.h>
 #include <src/App/InitApplication.h>
 
@@ -97,7 +97,7 @@ TEST_F(TestMaterial, TestMaterialsWithModel)
 
     // All LinearElastic models should be in IsotropicLinearElastic since it is inherited
     EXPECT_LE(materialsLinearElastic->size(), materials->size());
-    for (auto itp : *materialsLinearElastic) {
+    for (auto &itp : *materialsLinearElastic) {
         auto mat = itp.first;
         EXPECT_NO_THROW(materials->at(mat));
     }