]> git.tdb.fi Git - libs/datafile.git/commitdiff
Use metaprogramming constructs from std
authorMikko Rasa <tdb@tdb.fi>
Sun, 29 Aug 2021 14:17:23 +0000 (17:17 +0300)
committerMikko Rasa <tdb@tdb.fi>
Sun, 29 Aug 2021 14:17:23 +0000 (17:17 +0300)
source/collection.h
source/loader.h
source/loaderaction.h

index aa3c18a0fcc1a5e27f5d4ddc1278b58e65904877..473b58ed76329cf72d971f728d29ba0aae296760 100644 (file)
@@ -1,6 +1,7 @@
 #ifndef MSP_DATAFILE_COLLECTION_H_
 #define MSP_DATAFILE_COLLECTION_H_
 
 #ifndef MSP_DATAFILE_COLLECTION_H_
 #define MSP_DATAFILE_COLLECTION_H_
 
+#include <type_traits>
 #include <msp/core/attributes.h>
 #include <msp/core/maputils.h>
 #include <msp/core/noncopyable.h>
 #include <msp/core/attributes.h>
 #include <msp/core/maputils.h>
 #include <msp/core/noncopyable.h>
@@ -104,7 +105,7 @@ public:
                if(!item)
                        throw std::invalid_argument("Collection::add(item)");
 
                if(!item)
                        throw std::invalid_argument("Collection::add(item)");
 
-               typedef typename RemoveConst<T>::Type NCT;
+               typedef typename std::remove_cv<T>::type NCT;
                RefPtr<NCT> ptr(item);
                try
                {
                RefPtr<NCT> ptr(item);
                try
                {
@@ -122,7 +123,7 @@ public:
        template<typename T>
        T &get(const std::string &name) const
        {
        template<typename T>
        T &get(const std::string &name) const
        {
-               return extract<typename RemoveConst<T>::Type>(get_item(items, name));
+               return extract<typename std::remove_cv<T>::type>(get_item(items, name));
        }
 
        /** Gets a typed object from the collection.  If the name is not found,
        }
 
        /** Gets a typed object from the collection.  If the name is not found,
@@ -131,7 +132,7 @@ public:
        template<typename T>
        T &get(const std::string &name)
        {
        template<typename T>
        T &get(const std::string &name)
        {
-               typedef typename RemoveConst<T>::Type NCT;
+               typedef typename std::remove_cv<T>::type NCT;
                return extract<NCT>(get_var(name, get_type<NCT>(name)));
        }
 
                return extract<NCT>(get_var(name, get_type<NCT>(name)));
        }
 
@@ -142,13 +143,13 @@ public:
        T *find(const std::string &name) const
        {
                ItemMap::const_iterator i = items.find(name);
        T *find(const std::string &name) const
        {
                ItemMap::const_iterator i = items.find(name);
-               return (i!=items.end() ? extract<typename RemoveConst<T>::Type>(i->second) : 0);
+               return (i!=items.end() ? extract<typename std::remove_cv<T>::type>(i->second) : 0);
        }
 
        template<typename T>
        T *find(const std::string &name)
        {
        }
 
        template<typename T>
        T *find(const std::string &name)
        {
-               typedef typename RemoveConst<T>::Type NCT;
+               typedef typename std::remove_cv<T>::type NCT;
                const Variant *var = find_var(name, get_type<NCT>(name));
                return (var ? &extract<NCT>(*var) : 0);
        }
                const Variant *var = find_var(name, get_type<NCT>(name));
                return (var ? &extract<NCT>(*var) : 0);
        }
@@ -187,7 +188,7 @@ public:
        std::list<std::string> get_names() const
        {
                std::list<std::string> names;
        std::list<std::string> get_names() const
        {
                std::list<std::string> names;
-               gather_items<typename RemoveConst<T>::Type>(0, &names, 0, false);
+               gather_items<typename std::remove_cv<T>::type>(0, &names, 0, false);
                return names;
        }
 
                return names;
        }
 
@@ -197,7 +198,7 @@ public:
        std::list<std::string> get_names()
        {
                std::list<std::string> names;
        std::list<std::string> get_names()
        {
                std::list<std::string> names;
-               gather_items<typename RemoveConst<T>::Type>(0, &names, 0, true);
+               gather_items<typename std::remove_cv<T>::type>(0, &names, 0, true);
                return names;
        }
 
                return names;
        }
 
@@ -206,7 +207,7 @@ public:
        std::list<T *> get_list() const
        {
                std::vector<const Variant *> vars;
        std::list<T *> get_list() const
        {
                std::vector<const Variant *> vars;
-               gather_items<typename RemoveConst<T>::Type>(&vars, 0, 0, false);
+               gather_items<typename std::remove_cv<T>::type>(&vars, 0, 0, false);
                return extract_list<T>(vars);
        }
 
                return extract_list<T>(vars);
        }
 
@@ -215,12 +216,12 @@ public:
        template<typename T>
        std::list<T *> get_list()
        {
        template<typename T>
        std::list<T *> get_list()
        {
-               CollectionItemTypeBase *type = get_type<typename RemoveConst<T>::Type>();
+               CollectionItemTypeBase *type = get_type<typename std::remove_cv<T>::type>();
                if(type)
                        load_items_from_sources(*type);
 
                std::vector<const Variant *> vars;
                if(type)
                        load_items_from_sources(*type);
 
                std::vector<const Variant *> vars;
-               gather_items<typename RemoveConst<T>::Type>(&vars, 0, type, true);
+               gather_items<typename std::remove_cv<T>::type>(&vars, 0, type, true);
                return extract_list<T>(vars);
        }
 
                return extract_list<T>(vars);
        }
 
@@ -242,19 +243,19 @@ public:
        /// Checks whether a typed object exists in the collection.
        template<typename T>
        bool contains(const std::string &name) const
        /// Checks whether a typed object exists in the collection.
        template<typename T>
        bool contains(const std::string &name) const
-       { return get_status<typename RemoveConst<T>::Type>(name)==1; }
+       { return get_status<typename std::remove_cv<T>::type>(name)==1; }
 
        /** Checks whether a typed object exists in the collection or is loadable
        from a source. */
        template<typename T>
        bool contains(const std::string &name)
 
        /** Checks whether a typed object exists in the collection or is loadable
        from a source. */
        template<typename T>
        bool contains(const std::string &name)
-       { return get_status<typename RemoveConst<T>::Type>(name)>0; }
+       { return get_status<typename std::remove_cv<T>::type>(name)>0; }
 
        /// Returns the name of an item in the collection.
        template<typename T>
        const std::string &get_name(T *d) const
        {
 
        /// Returns the name of an item in the collection.
        template<typename T>
        const std::string &get_name(T *d) const
        {
-               typedef RefPtr<typename RemoveConst<T>::Type> RPNCT;
+               typedef RefPtr<typename std::remove_cv<T>::type> RPNCT;
 
                for(ItemMap::const_iterator i=items.begin(); i!=items.end(); ++i)
                        if(i->second.check_type<RPNCT>())
 
                for(ItemMap::const_iterator i=items.begin(); i!=items.end(); ++i)
                        if(i->second.check_type<RPNCT>())
index 574ecce0c11b5bac9af56d82b6a0782a5288d8b6..926a2a16a4adbf2d542701867a47358ad09a3467 100644 (file)
@@ -1,6 +1,7 @@
 #ifndef MSP_DATAFILE_LOADER_H_
 #define MSP_DATAFILE_LOADER_H_
 
 #ifndef MSP_DATAFILE_LOADER_H_
 #define MSP_DATAFILE_LOADER_H_
 
+#include <type_traits>
 #include <map>
 #include <msp/io/file.h>
 #include "loaderaction.h"
 #include <map>
 #include <msp/io/file.h>
 #include "loaderaction.h"
@@ -115,7 +116,7 @@ protected:
        /** Adds a keyword that is loaded by calling a function with a bound
        first argument. */
        template<typename L, typename B0, typename... Args>
        /** Adds a keyword that is loaded by calling a function with a bound
        first argument. */
        template<typename L, typename B0, typename... Args>
-       void add(const std::string &k, void (L::*func)(B0, Args...), const typename RemoveReference<B0>::Type &b0)
+       void add(const std::string &k, void (L::*func)(B0, Args...), const typename std::remove_reference<B0>::type &b0)
        { add(k, new LoaderFuncNBound1<L, B0, Args...>(func, b0)); }
 
        /** Adds a keyword that is loaded into a member of the loaded object. */
        { add(k, new LoaderFuncNBound1<L, B0, Args...>(func, b0)); }
 
        /** Adds a keyword that is loaded into a member of the loaded object. */
@@ -191,7 +192,7 @@ Loads an object from a file stored in a collection.  The object must havea
 public Loader class.  Any extra arguments are passed to the Loader constructor.
 */
 template<typename T, typename C, typename... Args>
 public Loader class.  Any extra arguments are passed to the Loader constructor.
 */
 template<typename T, typename C, typename... Args>
-typename EnableIf<NeedsCollection<typename T::Loader>::value, void>::No load(T &obj, C &coll, const std::string &fn, Args &... args)
+typename std::enable_if<!NeedsCollection<typename T::Loader>::value>::type load(T &obj, C &coll, const std::string &fn, Args &... args)
 {
        RefPtr<IO::Seekable> in = coll.open_raw(fn);
        if(!in)
 {
        RefPtr<IO::Seekable> in = coll.open_raw(fn);
        if(!in)
index c7c593829a1b24a3ccd161dad400f14b06d207e8..6931f4849ff21ff6fee354fb9ac904011422e284 100644 (file)
@@ -234,7 +234,7 @@ class LoaderFuncNBound1: public LoaderAction
 {
 protected:
        typedef void (L::*FuncType)(B0, Args...);
 {
 protected:
        typedef void (L::*FuncType)(B0, Args...);
-       typedef typename RemoveReference<B0>::Type Bound0Type;
+       typedef typename std::remove_reference<B0>::type Bound0Type;
 
        FuncType func;
        Bound0Type bound0;
 
        FuncType func;
        Bound0Type bound0;