Move all() up to IVariantLoader.
authorYann Dirson <ydirson@altern.org>
Sun, 20 Apr 2008 19:23:15 +0000 (20 21:23 +0200)
committerYann Dirson <ydirson@altern.org>
Sun, 20 Apr 2008 19:23:15 +0000 (20 21:23 +0200)
This may not be *required* for now, but all variant loaders must
implement it anyway - even though we call it manually on each sublass.

src/core/variantloader.h
src/variantfactories/kdepluginvariants.h

index c52c5b1..2f0eb59 100644 (file)
@@ -18,7 +18,12 @@ class Repository;
 class TAGUA_EXPORT IVariantLoader {
 public:
   virtual ~IVariantLoader();
-  
+
+  /**
+    * \return a list of all non-hidden variants.
+    */
+  virtual QStringList all() const = 0;
+
   virtual Repository* getRepository(const QString& variant) = 0;
 };
 
index af7fd63..1a21d6a 100644 (file)
@@ -28,11 +28,7 @@ class KDEPluginVariants : public IVariantLoader {
   Variant* create(const KService::Ptr& plugin);
 public:
   Variant* create(const QString& name);
-
-  /**
-    * \return a list of all non-hidden variants.
-    */
-  QStringList all() const;
+  virtual QStringList all() const;
   
   /**
     * \return the singleton IVariantLoader instance