From a3b5a07cf7c64f1a038839d3ef2f7b1d9fbf6c42 Mon Sep 17 00:00:00 2001
From: Josh Blum
Date: Sat, 27 Jul 2013 19:20:35 -0700
Subject: gras: added arg check with nice warning to callable and factory

---
 include/gras/detail/callable.hpp | 45 ++++++++++++++++++++--------------------
 include/gras/detail/factory.hpp  | 23 ++++++++++----------
 2 files changed, 35 insertions(+), 33 deletions(-)

(limited to 'include/gras/detail')

diff --git a/include/gras/detail/callable.hpp b/include/gras/detail/callable.hpp
index 68766fe..a2e9af4 100644
--- a/include/gras/detail/callable.hpp
+++ b/include/gras/detail/callable.hpp
@@ -16,6 +16,7 @@ struct GRAS_API CallableRegistryEntry
     CallableRegistryEntry(void);
     virtual ~CallableRegistryEntry(void);
     virtual PMCC call(const PMCC &args) = 0;
+    void arg_check(const PMCList &args, const size_t nargs);
 };
 
 /***********************************************************************
@@ -30,7 +31,7 @@ struct CallableRegistryEntryImpl0 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 0) throw a;
+        this->arg_check(a, 0);
         return PMC_M((_obj->*_fcn)());
     }
     ClassType *_obj; Fcn _fcn;
@@ -53,7 +54,7 @@ struct CallableRegistryEntryImplVoid0 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 0) throw a;
+        this->arg_check(a, 0);
         (_obj->*_fcn)(); return PMCC();
     }
     ClassType *_obj; Fcn _fcn;
@@ -79,7 +80,7 @@ struct CallableRegistryEntryImpl1 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 1) throw a;
+        this->arg_check(a, 1);
         return PMC_M((_obj->*_fcn)(a[0].safe_as<A0>()));
     }
     ClassType *_obj; Fcn _fcn;
@@ -102,7 +103,7 @@ struct CallableRegistryEntryImplVoid1 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 1) throw a;
+        this->arg_check(a, 1);
         (_obj->*_fcn)(a[0].safe_as<A0>()); return PMCC();
     }
     ClassType *_obj; Fcn _fcn;
@@ -128,7 +129,7 @@ struct CallableRegistryEntryImpl2 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 2) throw a;
+        this->arg_check(a, 2);
         return PMC_M((_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>()));
     }
     ClassType *_obj; Fcn _fcn;
@@ -151,7 +152,7 @@ struct CallableRegistryEntryImplVoid2 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 2) throw a;
+        this->arg_check(a, 2);
         (_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>()); return PMCC();
     }
     ClassType *_obj; Fcn _fcn;
@@ -177,7 +178,7 @@ struct CallableRegistryEntryImpl3 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 3) throw a;
+        this->arg_check(a, 3);
         return PMC_M((_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>()));
     }
     ClassType *_obj; Fcn _fcn;
@@ -200,7 +201,7 @@ struct CallableRegistryEntryImplVoid3 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 3) throw a;
+        this->arg_check(a, 3);
         (_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>()); return PMCC();
     }
     ClassType *_obj; Fcn _fcn;
@@ -226,7 +227,7 @@ struct CallableRegistryEntryImpl4 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 4) throw a;
+        this->arg_check(a, 4);
         return PMC_M((_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>()));
     }
     ClassType *_obj; Fcn _fcn;
@@ -249,7 +250,7 @@ struct CallableRegistryEntryImplVoid4 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 4) throw a;
+        this->arg_check(a, 4);
         (_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>()); return PMCC();
     }
     ClassType *_obj; Fcn _fcn;
@@ -275,7 +276,7 @@ struct CallableRegistryEntryImpl5 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 5) throw a;
+        this->arg_check(a, 5);
         return PMC_M((_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>()));
     }
     ClassType *_obj; Fcn _fcn;
@@ -298,7 +299,7 @@ struct CallableRegistryEntryImplVoid5 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 5) throw a;
+        this->arg_check(a, 5);
         (_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>()); return PMCC();
     }
     ClassType *_obj; Fcn _fcn;
@@ -324,7 +325,7 @@ struct CallableRegistryEntryImpl6 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 6) throw a;
+        this->arg_check(a, 6);
         return PMC_M((_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>(), a[5].safe_as<A5>()));
     }
     ClassType *_obj; Fcn _fcn;
@@ -347,7 +348,7 @@ struct CallableRegistryEntryImplVoid6 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 6) throw a;
+        this->arg_check(a, 6);
         (_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>(), a[5].safe_as<A5>()); return PMCC();
     }
     ClassType *_obj; Fcn _fcn;
@@ -373,7 +374,7 @@ struct CallableRegistryEntryImpl7 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 7) throw a;
+        this->arg_check(a, 7);
         return PMC_M((_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>(), a[5].safe_as<A5>(), a[6].safe_as<A6>()));
     }
     ClassType *_obj; Fcn _fcn;
@@ -396,7 +397,7 @@ struct CallableRegistryEntryImplVoid7 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 7) throw a;
+        this->arg_check(a, 7);
         (_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>(), a[5].safe_as<A5>(), a[6].safe_as<A6>()); return PMCC();
     }
     ClassType *_obj; Fcn _fcn;
@@ -422,7 +423,7 @@ struct CallableRegistryEntryImpl8 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 8) throw a;
+        this->arg_check(a, 8);
         return PMC_M((_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>(), a[5].safe_as<A5>(), a[6].safe_as<A6>(), a[7].safe_as<A7>()));
     }
     ClassType *_obj; Fcn _fcn;
@@ -445,7 +446,7 @@ struct CallableRegistryEntryImplVoid8 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 8) throw a;
+        this->arg_check(a, 8);
         (_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>(), a[5].safe_as<A5>(), a[6].safe_as<A6>(), a[7].safe_as<A7>()); return PMCC();
     }
     ClassType *_obj; Fcn _fcn;
@@ -471,7 +472,7 @@ struct CallableRegistryEntryImpl9 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 9) throw a;
+        this->arg_check(a, 9);
         return PMC_M((_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>(), a[5].safe_as<A5>(), a[6].safe_as<A6>(), a[7].safe_as<A7>(), a[8].safe_as<A8>()));
     }
     ClassType *_obj; Fcn _fcn;
@@ -494,7 +495,7 @@ struct CallableRegistryEntryImplVoid9 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 9) throw a;
+        this->arg_check(a, 9);
         (_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>(), a[5].safe_as<A5>(), a[6].safe_as<A6>(), a[7].safe_as<A7>(), a[8].safe_as<A8>()); return PMCC();
     }
     ClassType *_obj; Fcn _fcn;
@@ -520,7 +521,7 @@ struct CallableRegistryEntryImpl10 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 10) throw a;
+        this->arg_check(a, 10);
         return PMC_M((_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>(), a[5].safe_as<A5>(), a[6].safe_as<A6>(), a[7].safe_as<A7>(), a[8].safe_as<A8>(), a[9].safe_as<A9>()));
     }
     ClassType *_obj; Fcn _fcn;
@@ -543,7 +544,7 @@ struct CallableRegistryEntryImplVoid10 : CallableRegistryEntry
     PMCC call(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 10) throw a;
+        this->arg_check(a, 10);
         (_obj->*_fcn)(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>(), a[5].safe_as<A5>(), a[6].safe_as<A6>(), a[7].safe_as<A7>(), a[8].safe_as<A8>(), a[9].safe_as<A9>()); return PMCC();
     }
     ClassType *_obj; Fcn _fcn;
diff --git a/include/gras/detail/factory.hpp b/include/gras/detail/factory.hpp
index a2de116..ac4df6f 100644
--- a/include/gras/detail/factory.hpp
+++ b/include/gras/detail/factory.hpp
@@ -16,6 +16,7 @@ struct GRAS_API FactoryRegistryEntry
     FactoryRegistryEntry(void);
     virtual ~FactoryRegistryEntry(void);
     virtual Element *make(const PMCC &args) = 0;
+    void arg_check(const PMCList &args, const size_t nargs);
 };
 
 /***********************************************************************
@@ -29,7 +30,7 @@ struct FactoryRegistryEntryImpl0 : FactoryRegistryEntry
     Element *make(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 0) throw a;
+        this->arg_check(a, 0);
         return _fcn();
     }
     Fcn _fcn;
@@ -53,7 +54,7 @@ struct FactoryRegistryEntryImpl1 : FactoryRegistryEntry
     Element *make(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 1) throw a;
+        this->arg_check(a, 1);
         return _fcn(a[0].safe_as<A0>());
     }
     Fcn _fcn;
@@ -77,7 +78,7 @@ struct FactoryRegistryEntryImpl2 : FactoryRegistryEntry
     Element *make(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 2) throw a;
+        this->arg_check(a, 2);
         return _fcn(a[0].safe_as<A0>(), a[1].safe_as<A1>());
     }
     Fcn _fcn;
@@ -101,7 +102,7 @@ struct FactoryRegistryEntryImpl3 : FactoryRegistryEntry
     Element *make(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 3) throw a;
+        this->arg_check(a, 3);
         return _fcn(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>());
     }
     Fcn _fcn;
@@ -125,7 +126,7 @@ struct FactoryRegistryEntryImpl4 : FactoryRegistryEntry
     Element *make(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 4) throw a;
+        this->arg_check(a, 4);
         return _fcn(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>());
     }
     Fcn _fcn;
@@ -149,7 +150,7 @@ struct FactoryRegistryEntryImpl5 : FactoryRegistryEntry
     Element *make(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 5) throw a;
+        this->arg_check(a, 5);
         return _fcn(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>());
     }
     Fcn _fcn;
@@ -173,7 +174,7 @@ struct FactoryRegistryEntryImpl6 : FactoryRegistryEntry
     Element *make(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 6) throw a;
+        this->arg_check(a, 6);
         return _fcn(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>(), a[5].safe_as<A5>());
     }
     Fcn _fcn;
@@ -197,7 +198,7 @@ struct FactoryRegistryEntryImpl7 : FactoryRegistryEntry
     Element *make(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 7) throw a;
+        this->arg_check(a, 7);
         return _fcn(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>(), a[5].safe_as<A5>(), a[6].safe_as<A6>());
     }
     Fcn _fcn;
@@ -221,7 +222,7 @@ struct FactoryRegistryEntryImpl8 : FactoryRegistryEntry
     Element *make(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 8) throw a;
+        this->arg_check(a, 8);
         return _fcn(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>(), a[5].safe_as<A5>(), a[6].safe_as<A6>(), a[7].safe_as<A7>());
     }
     Fcn _fcn;
@@ -245,7 +246,7 @@ struct FactoryRegistryEntryImpl9 : FactoryRegistryEntry
     Element *make(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 9) throw a;
+        this->arg_check(a, 9);
         return _fcn(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>(), a[5].safe_as<A5>(), a[6].safe_as<A6>(), a[7].safe_as<A7>(), a[8].safe_as<A8>());
     }
     Fcn _fcn;
@@ -269,7 +270,7 @@ struct FactoryRegistryEntryImpl10 : FactoryRegistryEntry
     Element *make(const PMCC &args)
     {
         const PMCList &a = args.as<PMCList>();
-        if (a.size() < 10) throw a;
+        this->arg_check(a, 10);
         return _fcn(a[0].safe_as<A0>(), a[1].safe_as<A1>(), a[2].safe_as<A2>(), a[3].safe_as<A3>(), a[4].safe_as<A4>(), a[5].safe_as<A5>(), a[6].safe_as<A6>(), a[7].safe_as<A7>(), a[8].safe_as<A8>(), a[9].safe_as<A9>());
     }
     Fcn _fcn;
-- 
cgit