loplugin:constantfunction: vcl/unx/gtk3

Change-Id: Ieedf89bebb59e783dfe13ef2d5ae7ba3347cdc62

compilerplugins/clang/constantfunction.cxx

@@ -128,6 +128,10 @@ bool ConstantFunction::VisitFunctionDecl(const FunctionDecl * pFunctionDecl) {
     if (aFileName == SRCDIR "/filter/source/svg/svgreader.cxx") {
         return true;
     }
+    // vcl/unx/gtk3 re-using vcl/unx/gtk:
+    if (aFileName.find("/../../gtk/") != std::string::npos) {
+        return true;
+    }
     // used by code generated by python
     if (getFilename(pFunctionDecl->getCanonicalDecl()->getLocStart()) == SRCDIR "/writerfilter/source/ooxml/OOXMLFastContextHandler.hxx") {
         return true;
@@ -400,6 +404,12 @@ bool ConstantFunction::VisitFunctionDecl(const FunctionDecl * pFunctionDecl) {
     if (aFunctionName == "SwVectorModifyBase::dumpAsXml") {
         return true;
     }
+    // vcl/unx/gtk3 re-using vcl/unx/gtk:
+    if (aFunctionName == "GtkData::initNWF"
+        || aFunctionName == "GtkSalFrame::EnsureAppMenuWatch")
+    {
+        return true;
+    }
 
     // can't mess with the TYPEINIT macros in include/tools/rtti.hxx or the LINK macros in include/tools/link.hxx
     std::string aImmediateMacro = "";

vcl/inc/unx/gtk/gtkdata.hxx

@@ -176,9 +176,7 @@ public:
 
     virtual void PostUserEvent() SAL_OVERRIDE;
 
-#if GTK_CHECK_VERSION(3,0,0)
-    guint32 GetLastUserEventTime( bool /* b */ ) { return GDK_CURRENT_TIME; } // horrible hack
-#else
+#if !GTK_CHECK_VERSION(3,0,0)
     virtual bool Dispatch( XEvent *pEvent ) SAL_OVERRIDE;
 #endif
 };