]> git.tdb.fi Git - libs/core.git/commitdiff
Mark constructors and destructors as default where appropriate
authorMikko Rasa <tdb@tdb.fi>
Sun, 31 Oct 2021 11:28:41 +0000 (13:28 +0200)
committerMikko Rasa <tdb@tdb.fi>
Sun, 31 Oct 2021 18:27:41 +0000 (20:27 +0200)
23 files changed:
source/core/application.h
source/core/getopt.h
source/core/maputils.h
source/core/refptr.h
source/core/systemerror.h
source/core/variant.h
source/fs/dir.h
source/fs/path.cpp
source/fs/path.h
source/fs/stat.h
source/fs/stat_private.h
source/io/file.h
source/io/mode.h
source/io/seekable.h
source/io/zlibcompressed.h
source/stringcodec/codec.h
source/stringcodec/except.h
source/strings/fmt.h
source/strings/lexicalcast.h
source/strings/regex.h
source/strings/regmatch.h
source/time/timedelta.h
source/time/timestamp.h

index 40215ef964aff63965d50cfb3fdd55f3fc4a3cd1..8ff286f3e2cc3202de01d85247a3126191128cbf 100644 (file)
@@ -18,7 +18,7 @@ protected:
        protected:
                Starter();
        public:
        protected:
                Starter();
        public:
-               virtual ~Starter() { }
+               virtual ~Starter() = default;
 
                virtual Application *create_app(int, char **) = 0;
        };
 
                virtual Application *create_app(int, char **) = 0;
        };
@@ -38,7 +38,7 @@ private:
 protected:
        Application(const std::string & = std::string());
 public:
 protected:
        Application(const std::string & = std::string());
 public:
-       virtual ~Application() { }
+       virtual ~Application() = default;
 
        /** Constructs an instance of the registered application class and runs it.
        If the application throws a usage_error, a help message is printed.  The
 
        /** Constructs an instance of the registered application class and runs it.
        If the application throws a usage_error, a help message is printed.  The
index a1208f4c1fa9e423314225bb87e8fc91903149e8..5c21ccb97bb7142aeb2b968ebd0e8e23e1303676 100644 (file)
@@ -17,7 +17,7 @@ private:
 
 public:
        usage_error(const std::string &w, const std::string &h = std::string()): std::runtime_error(w), help_(h) { }
 
 public:
        usage_error(const std::string &w, const std::string &h = std::string()): std::runtime_error(w), help_(h) { }
-       ~usage_error() throw() { }
+       virtual ~usage_error() throw() = default;
 
        const char *help() const throw() { return help_.c_str(); }
 };
 
        const char *help() const throw() { return help_.c_str(); }
 };
@@ -72,9 +72,9 @@ public:
        class Option
        {
        protected:
        class Option
        {
        protected:
-               Option() { }
+               Option() = default;
        public:
        public:
-               virtual ~Option() { }
+               virtual ~Option() = default;
 
                /// Sets help text for the option.
                virtual Option &set_help(const std::string &) = 0;
 
                /// Sets help text for the option.
                virtual Option &set_help(const std::string &) = 0;
@@ -92,9 +92,9 @@ public:
        class Argument
        {
        protected:
        class Argument
        {
        protected:
-               Argument() { }
+               Argument() = default;
        public:
        public:
-               virtual ~Argument() { }
+               virtual ~Argument() = default;
 
                virtual Argument &set_help(const std::string &) = 0;
        };
 
                virtual Argument &set_help(const std::string &) = 0;
        };
@@ -103,9 +103,9 @@ private:
        class Store
        {
        protected:
        class Store
        {
        protected:
-               Store() { }
+               Store() = default;
        public:
        public:
-               virtual ~Store() { }
+               virtual ~Store() = default;
 
                virtual Store *clone() const = 0;
 
 
                virtual Store *clone() const = 0;
 
index 88f47cd367b60c3e85d72d075e59370971295c79..7eae13471accbff5f67803bb1818af6c78b052f5 100644 (file)
@@ -47,7 +47,7 @@ public:
                runtime_error(make_what(typeid(T), MapUtilsInternal::stringify_key(k)))
        { }
 
                runtime_error(make_what(typeid(T), MapUtilsInternal::stringify_key(k)))
        { }
 
-       virtual ~key_error() throw() { }
+       virtual ~key_error() throw() = default;
 
 private:
        static std::string make_what(const std::type_info &, const std::string &);
 
 private:
        static std::string make_what(const std::type_info &, const std::string &);
index 6285d0ef4031e2e1bc29a1a0495b3e67bc5721dc..21da15df6a21d2581f640e88245c969da5eb0f80 100644 (file)
@@ -35,7 +35,7 @@ private:
        RefCounts *counts = 0;
 
 public:
        RefCounts *counts = 0;
 
 public:
-       RefPtr() { }
+       RefPtr() = default;
        RefPtr(T *d): data(d), counts(data ? new RefCounts : 0) { incref(); }
 private:
        RefPtr(T *d, RefCounts *c): data(d), counts(d ? c : 0) { incref(); }
        RefPtr(T *d): data(d), counts(data ? new RefCounts : 0) { incref(); }
 private:
        RefPtr(T *d, RefCounts *c): data(d), counts(d ? c : 0) { incref(); }
@@ -106,7 +106,7 @@ private:
        RefCounts *counts = 0;
 
 public:
        RefCounts *counts = 0;
 
 public:
-       WeakPtr() { }
+       WeakPtr() = default;
 private:
        WeakPtr(T *d, RefCounts *c): data(d), counts(d ? c : 0) { incref(); }
 
 private:
        WeakPtr(T *d, RefCounts *c): data(d), counts(d ? c : 0) { incref(); }
 
index c6b516af9f5dddba7d435a6d02408a9f68820f0e..d0a5634cf38e04cc5d70e2df3e677c622878ed97 100644 (file)
@@ -14,7 +14,7 @@ private:
 public:
        system_error(const std::string &, int = -1);
        system_error(const std::string &, const std::string &);
 public:
        system_error(const std::string &, int = -1);
        system_error(const std::string &, const std::string &);
-       virtual ~system_error() throw() { }
+       virtual ~system_error() throw() = default;
 
        int code() const throw() { return code_; }
 
 
        int code() const throw() { return code_; }
 
index 05a8139c679783000f9e56d65ae6674029931d49..1ad086e70381254422d92330e277b7fd43f4826d 100644 (file)
@@ -12,7 +12,7 @@ class type_mismatch: public std::runtime_error
 {
 public:
        type_mismatch(const std::type_info &, const std::type_info &);
 {
 public:
        type_mismatch(const std::type_info &, const std::type_info &);
-       ~type_mismatch() throw() { }
+       virtual ~type_mismatch() throw() = default;
 };
 
 
 };
 
 
@@ -53,7 +53,7 @@ private:
        StoreBase *store = 0;
 
 public:
        StoreBase *store = 0;
 
 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) { }
        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) { }
index c0a1c7460324f3163715d5c119b0fea701b3517e..fa2793a15532f04943a5d827ac8e4647c766e390 100644 (file)
@@ -13,7 +13,7 @@ class not_a_directory: public std::runtime_error
 {
 public:
        not_a_directory(const Path &);
 {
 public:
        not_a_directory(const Path &);
-       virtual ~not_a_directory() throw() { }
+       virtual ~not_a_directory() throw() = default;
 };
 
 /// Creates a directory
 };
 
 /// Creates a directory
index adf376f7a0b0db57341370a7bd07d61b643a7c04..0e86600453be225bcb647ff699f3cb0b5f20649b 100644 (file)
@@ -17,9 +17,6 @@ inline bool is_windows_drive(const string &p)
 namespace Msp {
 namespace FS {
 
 namespace Msp {
 namespace FS {
 
-Path::Path()
-{ }
-
 Path::Path(const string &p)
 {
        if(p.empty())
 Path::Path(const string &p)
 {
        if(p.empty())
index 9cc0dc243e32900bf869dc92df85a1ddea268251..30fe683eabafff9f64b8841f8706ea4cde8be341 100644 (file)
@@ -70,7 +70,7 @@ private:
        PositionArray separators;
 
 public:
        PositionArray separators;
 
 public:
-       Path();
+       Path() = default;
        Path(const std::string &);
        Path(const char *p): Path(std::string(p)) { }
 
        Path(const std::string &);
        Path(const char *p): Path(std::string(p)) { }
 
index 00e3eb939dc5000b5d99299a40d8b4bd0e745a3a..d23a8e82ef6376c520211c16e0828dd42ecd7b73 100644 (file)
@@ -42,7 +42,7 @@ private:
        Private *priv = 0;
 
 public:
        Private *priv = 0;
 
 public:
-       Stat() { }
+       Stat() = default;
        Stat(const Stat &);
        Stat &operator=(const Stat &);
        ~Stat();
        Stat(const Stat &);
        Stat &operator=(const Stat &);
        ~Stat();
index 60438a269717373f9cc418aaa3266ba9b57df4e8..a6da47351cfcd497ee2136aaae17fdfb881353b6 100644 (file)
@@ -11,7 +11,7 @@ struct Stat::Private
        UserID owner_id = INVALID_UID;
        GroupID group_id = INVALID_GID;
 
        UserID owner_id = INVALID_UID;
        GroupID group_id = INVALID_GID;
 
-       Private();
+       Private() = default;
        Private(const Private &);
        ~Private();
 
        Private(const Private &);
        ~Private();
 
index aeac6c8d265556974ddfee94febdb8ea5fd131b7..36f37706a8432b35aed12f8772b9bd3958c7a78b 100644 (file)
@@ -15,14 +15,14 @@ class file_not_found: public std::runtime_error
 {
 public:
        file_not_found(const std::string &fn): std::runtime_error(fn) { }
 {
 public:
        file_not_found(const std::string &fn): std::runtime_error(fn) { }
-       ~file_not_found() throw() { }
+       virtual ~file_not_found() throw() = default;
 };
 
 class file_already_exists: public std::runtime_error
 {
 public:
        file_already_exists(const std::string &fn): std::runtime_error(fn) { }
 };
 
 class file_already_exists: public std::runtime_error
 {
 public:
        file_already_exists(const std::string &fn): std::runtime_error(fn) { }
-       ~file_already_exists() throw() { }
+       virtual ~file_already_exists() throw() = default;
 };
 
 
 };
 
 
index 0f7bb4fc37650cabe29389021d9c06be0b64bebe..a643a7f32074782bb863c99959d6d14e1a66f496 100644 (file)
@@ -34,7 +34,7 @@ class invalid_access: public std::logic_error
 {
 public:
        invalid_access(Mode);
 {
 public:
        invalid_access(Mode);
-       ~invalid_access() throw() { }
+       virtual ~invalid_access() throw() = default;
 };
 
 } // namespace IO
 };
 
 } // namespace IO
index 99915d850efc3a8879d88a7e9e3e9de7e11eb531..68454d038bc31ff50118044292906af36dba6895 100644 (file)
@@ -24,14 +24,14 @@ class bad_seek: public std::runtime_error
 {
 public:
        bad_seek(SeekOffset, SeekType);
 {
 public:
        bad_seek(SeekOffset, SeekType);
-       virtual ~bad_seek() throw() { }
+       virtual ~bad_seek() throw() = default;
 };
 
 
 class Seekable: public Base
 {
 protected:
 };
 
 
 class Seekable: public Base
 {
 protected:
-       Seekable() { }
+       Seekable() = default;
 
 public:
        /** Changes the read/write offset.  Returns the new offset. */
 
 public:
        /** Changes the read/write offset.  Returns the new offset. */
index f31bb2385a28a7dc7c40ef4432501ee89cae8661..fb12d61a4eff0fdcb07386582b759c302fc361dd 100644 (file)
@@ -16,7 +16,7 @@ private:
 
 public:
        zlib_error(const std::string &, int);
 
 public:
        zlib_error(const std::string &, int);
-       ~zlib_error() throw() { }
+       virtual ~zlib_error() throw() = default;
 
        int code() const throw() { return code_; }
 };
 
        int code() const throw() { return code_; }
 };
index efa165784ac2988f674c0c6be5794c3a1a44c63d..f434ed82e597c706b7bf54de4e24081b4447f693 100644 (file)
@@ -43,7 +43,7 @@ public:
 
                Encoder(ErrorMode em): err_mode(em==DEFAULT ? THROW_ON_ERROR : em) { }
        public:
 
                Encoder(ErrorMode em): err_mode(em==DEFAULT ? THROW_ON_ERROR : em) { }
        public:
-               virtual ~Encoder() { }
+               virtual ~Encoder() = default;
 
                /** Encodes a single unicode character.  If the character can't be
                represented in this encoding, error() should be called. */
 
                /** Encodes a single unicode character.  If the character can't be
                represented in this encoding, error() should be called. */
@@ -94,7 +94,7 @@ public:
 
                Decoder(ErrorMode em): err_mode(em==DEFAULT ? THROW_ON_ERROR : em) { }
        public:
 
                Decoder(ErrorMode em): err_mode(em==DEFAULT ? THROW_ON_ERROR : em) { }
        public:
-               virtual ~Decoder() { }
+               virtual ~Decoder() = default;
 
                /** Decodes a single character from a string.  The iterator is advanced
                to the next character.  For stateful codecs, -1 may be returned if a
 
                /** Decodes a single character from a string.  The iterator is advanced
                to the next character.  For stateful codecs, -1 may be returned if a
@@ -131,9 +131,9 @@ public:
        };
 
 protected:
        };
 
 protected:
-       Codec() { }
+       Codec() = default;
 public:
 public:
-       virtual ~Codec() { }
+       virtual ~Codec() = default;
 
        /** Returns the name of the encoding handled by this codec. */
        virtual const char *get_name() const = 0;
 
        /** Returns the name of the encoding handled by this codec. */
        virtual const char *get_name() const = 0;
index 6e2566682669ec0bbdc6a925d4fca2fb653c2250..a1126e71f13286c6d3accf0650e79fe8b6d13e07 100644 (file)
@@ -14,7 +14,7 @@ class codec_error: public std::runtime_error
 {
 public:
        codec_error(const std::string &w): std::runtime_error(w) { }
 {
 public:
        codec_error(const std::string &w): std::runtime_error(w) { }
-       virtual ~codec_error() throw() { }
+       virtual ~codec_error() throw() = default;
 };
 
 
 };
 
 
@@ -25,7 +25,7 @@ class invalid_character: public codec_error
 {
 public:
        invalid_character(unichar, const std::string &);
 {
 public:
        invalid_character(unichar, const std::string &);
-       virtual ~invalid_character() throw() { }
+       virtual ~invalid_character() throw() = default;
 };
 
 
 };
 
 
@@ -36,7 +36,7 @@ class invalid_sequence: public codec_error
 {
 public:
        invalid_sequence(const std::string::const_iterator &, const std::string::const_iterator &, const std::string &);
 {
 public:
        invalid_sequence(const std::string::const_iterator &, const std::string::const_iterator &, const std::string &);
-       virtual ~invalid_sequence() throw() { }
+       virtual ~invalid_sequence() throw() = default;
 
 private:
        std::string format_sequence(const std::string::const_iterator &, const std::string::const_iterator &);
 
 private:
        std::string format_sequence(const std::string::const_iterator &, const std::string::const_iterator &);
index 98e7e64dd111582d2532aa21adf7efc28e758183..3c1db9d3e3c37c14f76ad6a7cba166925f00a26f 100644 (file)
@@ -11,7 +11,7 @@ class format_error: public std::logic_error
 {
 public:
        format_error(const std::string &w): std::logic_error(w) { }
 {
 public:
        format_error(const std::string &w): std::logic_error(w) { }
-       virtual ~format_error() throw() { }
+       virtual ~format_error() throw() = default;
 };
 
 
 };
 
 
index 5d580806631f3c4c7fc964d39d30f065896f783f..0c5f0a34caa38844be7265c8366e9be34ea1ea47 100644 (file)
@@ -16,7 +16,7 @@ class lexical_error: public std::runtime_error
 {
 public:
        lexical_error(const std::string &w): runtime_error(w) { }
 {
 public:
        lexical_error(const std::string &w): runtime_error(w) { }
-       virtual ~lexical_error() throw() { }
+       virtual ~lexical_error() throw() = default;
 };
 
 
 };
 
 
@@ -27,7 +27,7 @@ class format_mismatch: public lexical_error
 {
 public:
        format_mismatch(const std::string &w): lexical_error(w) { }
 {
 public:
        format_mismatch(const std::string &w): lexical_error(w) { }
-       virtual ~format_mismatch() throw() { }
+       virtual ~format_mismatch() throw() = default;
 };
 
 
 };
 
 
index e830c5bbda253cb111e7e3e07199707c8b884e73..ed53009f920cf6b341d1891dbcc6d89ab3229aa6 100644 (file)
@@ -11,7 +11,7 @@ class bad_regex: public std::logic_error
 {
 public:
        bad_regex(const std::string &, const std::string &, const std::string::const_iterator &);
 {
 public:
        bad_regex(const std::string &, const std::string &, const std::string::const_iterator &);
-       virtual ~bad_regex() throw() { }
+       virtual ~bad_regex() throw() = default;
 
 private:
        std::string make_where(const std::string &, const std::string::const_iterator &);
 
 private:
        std::string make_where(const std::string &, const std::string::const_iterator &);
index 0858e5a8dfc5cec44afba0d308ca45ef3c8d3206..48f65c6df2b2409403762d0a28f067c8c297b07b 100644 (file)
@@ -40,7 +40,7 @@ private:
 
 public:
        /** Constructs a RegMatch representing a non-match. */
 
 public:
        /** Constructs a RegMatch representing a non-match. */
-       RegMatch() { }
+       RegMatch() = default;
 
        /** Constructs a new RegMatch from a string and groups.  The length and str
        members of each group are computed and need not be set.  Intended to be used
 
        /** Constructs a new RegMatch from a string and groups.  The length and str
        members of each group are computed and need not be set.  Intended to be used
index 332683533cad5e3fea5e076360bb50fb2fa1714c..a3e62f161c33684f13b90444e23b40fda02567f4 100644 (file)
@@ -19,7 +19,7 @@ private:
 
 public:
        /** Constructs a zero TimeDelta. */
 
 public:
        /** Constructs a zero TimeDelta. */
-       TimeDelta() { }
+       TimeDelta() = default;
 
        /** Constructs a TimeDelta from a plain number.  The purpose of this is to
        allow serialization together with the raw() function. */
 
        /** Constructs a TimeDelta from a plain number.  The purpose of this is to
        allow serialization together with the raw() function. */
index d16bb06498c957e18ee34fe12ae2e7bdad149033..2a5c4e1b73bb85df7b9dc9bd824cdbdf46bc8632 100644 (file)
@@ -21,7 +21,7 @@ private:
 public:
        /** Construct a TimeStamp that represents an arbitarily distant point in the
        past.  It's guaranteed to be less than any valid timestamp. */
 public:
        /** Construct a TimeStamp that represents an arbitarily distant point in the
        past.  It's guaranteed to be less than any valid timestamp. */
-       TimeStamp() { }
+       TimeStamp() = default;
 
        /** Constructs a TimeStamp from a plain number.  The purpose of this is to allow
        serialization together with the raw() function. */
 
        /** Constructs a TimeStamp from a plain number.  The purpose of this is to allow
        serialization together with the raw() function. */