]> git.tdb.fi Git - libs/core.git/blobdiff - source/core/variant.h
Use nullptr instead of 0 for pointers
[libs/core.git] / source / core / variant.h
index 05a8139c679783000f9e56d65ae6674029931d49..443befb481620b4242e178a4b377192179af1adc 100644 (file)
@@ -12,7 +12,7 @@ class type_mismatch: public std::runtime_error
 {
 public:
        type_mismatch(const std::type_info &, const std::type_info &);
-       ~type_mismatch() throw() { }
+       virtual ~type_mismatch() throw() = default;
 };
 
 
@@ -39,24 +39,24 @@ private:
                virtual const std::type_info &type_id() const { return typeid(T); }
                virtual StoreBase *clone() const { return new Store<T>(data); }
                virtual bool type_equals(const StoreBase &s) const { return dynamic_cast<const Store<T> *>(&s); }
-               virtual bool value_equals(const StoreBase &s) const { return value_equals_<T>(s); }
+               virtual bool value_equals(const StoreBase &s) const { return _value_equals<T>(s); }
 
                template<typename U>
-               typename std::enable_if<IsEqualityComparable<U>::value, bool>::type value_equals_(const StoreBase &s) const
+               typename std::enable_if<IsEqualityComparable<U>::value, bool>::type _value_equals(const StoreBase &s) const
                { const Store<T> *t = dynamic_cast<const Store<T> *>(&s); return (t && t->data==data); }
 
                template<typename U>
-               typename std::enable_if<!IsEqualityComparable<U>::value, bool>::type value_equals_(const StoreBase &) const
+               typename std::enable_if<!IsEqualityComparable<U>::value, bool>::type _value_equals(const StoreBase &) const
                { return false; }
        };
 
-       StoreBase *store = 0;
+       StoreBase *store = nullptr;
 
 public:
-       Variant() { }
+       Variant() = default;
        template<typename T>
        Variant(const T &v): store(new Store<typename std::remove_cv<T>::type>(v)) { }
-       Variant(const Variant &v): store(v.store ? v.store->clone() : 0) { }
+       Variant(const Variant &v): store(v.store ? v.store->clone() : nullptr) { }
        ~Variant() { delete store; }
 
        template<typename T>
@@ -70,7 +70,7 @@ public:
        Variant &operator=(const Variant &v)
        {
                delete store;
-               store = (v.store ? v.store->clone() : 0);
+               store = (v.store ? v.store->clone() : nullptr);
                return *this;
        }
 
@@ -101,7 +101,7 @@ public:
        template<typename T>
        bool check_type() const
        {
-               return dynamic_cast<Store<typename std::remove_cv<T>::type> *>(store)!=0;
+               return dynamic_cast<Store<typename std::remove_cv<T>::type> *>(store);
        }
 
        bool check_same_type(const Variant &v) const