Streamline application class registration

diff --git a/source/core/application.cpp b/source/core/application.cpp
index 17611c1429ac2ef716c1687ba892b3b6356e6f74..a588e9dbb7900cff2b3d15a200f41a9be2382d8f 100644 (file)
--- a/source/core/application.cpp
+++ b/source/core/application.cpp
@@ -40,9 +40,9 @@ int Application::run(int argc, char **argv, void *data)
        }
        called=true;
 
        }
        called=true;
 

-       if(!reg_app_)
+       if(!starter_)

        {
        {

-               cerr<<"Trying to run with no application class registered!\n";
+               cerr<<"Trying to run with no RegisteredApplication class!\n";

                return 126;
        }
 
                return 126;
        }
 


@@ -52,11 +52,11 @@ int Application::run(int argc, char **argv, void *data)
        {
                try
                {
        {
                try
                {

-                       app_=reg_app_->create_app(argc, argv);
+                       app_=starter_->create_app(argc, argv);

                }
                catch(const UsageError &e)
                {
                }
                catch(const UsageError &e)
                {

-                       reg_app_->usage(e.what(), argv[0], e.get_brief());
+                       starter_->usage(e.what(), argv[0], e.get_brief());

                        return 1;
                }
 
                        return 1;
                }
 


@@ -144,15 +144,12 @@ void Application::sighandler_(int s)
 }
 
 
 }
 
 

-Application::RegBase::RegBase()
+Application::Starter::Starter()

 {
 {

-       if(reg_app_)
-       {
-               cerr<<"Warning: registering the application twice\n";
-               delete reg_app_;
-       }
+       if(starter_)
+               throw InvalidState("Can't create more than one Starter instance");

 
 

-       reg_app_=this;
+       starter_=this;

 }
 
 Application *Application::app_=0;
 }
 
 Application *Application::app_=0;

diff --git a/source/core/application.h b/source/core/application.h
index 38df732f1a5779782566d249b701a19ca99011a2..305fbfe34661967a3d8dbeca46ed30f2f9c1596f 100644 (file)
--- a/source/core/application.h
+++ b/source/core/application.h
@@ -17,29 +17,22 @@ member of type RegApp<MainClass>.
 class Application
 {
 protected:
 class Application
 {
 protected:

-       class RegBase
+       class Starter

        {
        {

-       public:
-               virtual Application *create_app(int, char **)=0;
-               virtual void usage(const char *, const char *, bool)=0;
-               virtual ~RegBase() { }

        protected:
        protected:

-               RegBase();
-       };
-
-       template<typename T>
-       class RegApp: public RegBase
-       {
+               Starter();

        public:
        public:

-               Application *create_app(int argc, char **argv) { return new T(argc, argv); }
-               void usage(const char *r, const char *a, bool b) { T::usage(r, a, b); }
+               virtual ~Starter() { }
+
+               virtual Application *create_app(int, char **) = 0;
+               virtual void usage(const char *, const char *, bool) = 0;

        };
 
        bool done;
        int exit_code;
 
 private:
        };
 
        bool done;
        int exit_code;
 
 private:

-       static RegBase *reg_app_;
+       static Starter *starter_;

        static Application *app_;
        static void *data_;
 
        static Application *app_;
        static void *data_;
 


@@ -60,11 +53,32 @@ protected:
        virtual void sighandler(int) { }
 private:
        static void sighandler_(int);
        virtual void sighandler(int) { }
 private:
        static void sighandler_(int);

+};

 
        Application(const Application &);
        Application &operator=(const Application &);
 
        Application(const Application &);
        Application &operator=(const Application &);

+
+template<typename T>
+class RegisteredApplication: public Application
+{
+private:
+       class Starter: public Application::Starter
+       {
+       public:
+               Application *create_app(int argc, char **argv) { return new T(argc, argv); }
+               void usage(const char *r, const char *a, bool b) { T::usage(r, a, b); }
+       };
+
+       static Starter starter_;
+
+protected:
+       // Force the starter into existence
+       RegisteredApplication() { (void)starter_; }

 };
 
 };
 

+template<typename T>
+typename RegisteredApplication<T>::Starter RegisteredApplication<T>::starter_;
+

 } // namespace Msp
 
 #endif
 } // namespace Msp
 
 #endif