[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdevelop/lib/interfaces
From:       Andras Mantia <amantia () kde ! org>
Date:       2007-03-28 13:54:56
Message-ID: 1175090096.405731.18208.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 647489 by amantia:

Do such functions really need to implemented inline? :-(

 M  +0 -9      iplugincontroller.cpp  
 M  +8 -2      iplugincontroller.h  


--- trunk/KDE/kdevelop/lib/interfaces/iplugincontroller.cpp #647488:647489
@@ -41,15 +41,6 @@
 {
 }
 
-template<class Extension> Extension* IPluginController::extensionForPlugin( const \
                QString &extension, const QString &pluginname)
-{
-    IPlugin *plugin = pluginForExtension(extension, pluginname);
-    if (plugin)
-        return plugin->extension<Extension*>();
-    else
-        return 0L;
-}
-
 KPluginInfo::List IPluginController::query( const QString &serviceType,
         const QString &constraint )
 {
--- trunk/KDE/kdevelop/lib/interfaces/iplugincontroller.h #647488:647489
@@ -31,13 +31,13 @@
 #include <kurl.h>
 #include <kservice.h>
 
+#include "iplugin.h"
 #include "kdevexport.h"
 
 class QExtensionManager;
 
 namespace KDevelop
 {
-class IPlugin;
 class ProfileEngine;
 
 /**
@@ -110,7 +110,13 @@
      * @param pluginname The name of the plugin to load if multiple plugins for the \
                extension exist, corresponds to the X-KDE-PluginInfo-Name
      * @return Pointer to the extension interface or 0 if no plugin supports it
       */
-    template<class Extension> Extension* extensionForPlugin( const QString \
&extension, const QString &pluginname); +    template<class Extension> Extension* \
extensionForPlugin( const QString &extension, const QString &pluginname) { +        \
IPlugin *plugin = pluginForExtension(extension, pluginname); +        if (plugin)
+            return plugin->extension<Extension>();
+        else
+            return 0L;
+    }
 
 
     /**


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic