CPPcheck fixes

parser/context.cc

@@ -611,7 +611,6 @@ QString Context::removeCommonTrailingSpaces(const QString& in) {
         // no content
         return QString("");
     } else {
-        QString placeHolder = "";
 
         for (int line=0;line<lines.length();line++) {
             int subLength = lines[line].length() - commonTrailingSpaces;

types/objectId/TreeItem.cc

@@ -45,7 +45,7 @@
 
 //--------------------------------------------------------------------------------
 
-TreeItem::TreeItem(int _id, QString _name, DataType _type, TreeItem* _parent) :
+TreeItem::TreeItem(int _id, const QString& _name, DataType _type, TreeItem* _parent) :
   id_(_id),
   dataType_(_type),
   visible_(true),
@@ -126,7 +126,7 @@ QString TreeItem::name() {
 
 //--------------------------------------------------------------------------------
 
-void TreeItem::name(QString _name ) {
+void TreeItem::name(const QString& _name ) {
   name_ = _name;
 }
 

types/objectId/TreeItem.hh

@@ -54,7 +54,7 @@ class TreeItem {
 
   public :
 
-    TreeItem(int _id, QString _name, DataType _type, TreeItem* _parent);
+    TreeItem(int _id, const QString& _name, DataType _type, TreeItem* _parent);
 
   // static members
   public:
@@ -82,7 +82,7 @@ class TreeItem {
 
     /// name
     QString name( );
-    void name( QString _name );
+    void name( const QString& _name );
 
   private:
     bool visible_;