gcc/clang choke on inline in cpp (which VS puts there for some reas…

…on when asked to create an implementation)

src/openms_gui/include/OpenMS/VISUAL/TOPPASTreeView.h

@@ -39,7 +39,7 @@ namespace OpenMS
     /// Filter tree elements by name (case insensitive and partial=substring matches are valid)
     /// An empty filter shows all elements.
     /// If an element in a subtree is matched, all parents up to the root are also shown.
-    void filter(QString must_match);
+    void filter(const QString& must_match);
 
     /// expand all subtrees, i.e. make them visible
     void expandAll();

src/openms_gui/source/VISUAL/TOPPASTreeView.cpp

@@ -27,10 +27,7 @@ namespace OpenMS
 
   TOPPASTreeView::~TOPPASTreeView() = default;
 
-  /// Filter tree elements by name
-  /// An empty filter shows all elements
-
-  inline void TOPPASTreeView::filter(QString must_match)
+  void TOPPASTreeView::filter(const QString& must_match)
   {
     // hide all
     QTreeWidgetItemIterator it(this);
@@ -88,7 +85,7 @@ namespace OpenMS
     }
   }
 
-  void TOPPASTreeView::mousePressEvent(QMouseEvent * event)
+  void TOPPASTreeView::mousePressEvent(QMouseEvent* event)
   {
     QTreeWidget::mousePressEvent(event);
 
@@ -98,7 +95,7 @@ namespace OpenMS
     }
   }
 
-  void TOPPASTreeView::mouseMoveEvent(QMouseEvent * event)
+  void TOPPASTreeView::mouseMoveEvent(QMouseEvent* event)
   {
     QTreeWidget::mouseMoveEvent(event);
 
@@ -126,7 +123,7 @@ namespace OpenMS
     drag->exec(Qt::CopyAction);
   }
 
-  void TOPPASTreeView::keyPressEvent(QKeyEvent * e)
+  void TOPPASTreeView::keyPressEvent(QKeyEvent* e)
   {
     QTreeWidget::keyPressEvent(e);
     if (currentItem() && e->key() == Qt::Key_Return)
@@ -140,14 +137,13 @@ namespace OpenMS
     }
   }
 
-  void TOPPASTreeView::enterEvent(QEvent * /*e*/)
+  void TOPPASTreeView::enterEvent(QEvent* /*e*/)
   {
     setFocus();
   }
 
-  void TOPPASTreeView::leaveEvent(QEvent * /*e*/)
+  void TOPPASTreeView::leaveEvent(QEvent* /*e*/)
   {
-
   }
 
 } //namespace OpenMS