Przeglądaj źródła

entity: implicit noexcept-ness for dtors

Michele Caini 1 rok temu
rodzic
commit
044bfeefba

+ 1 - 1
src/entt/entity/group.hpp

@@ -93,7 +93,7 @@ template<typename... Lhs, typename... Rhs>
 
 struct group_descriptor {
     using size_type = std::size_t;
-    virtual ~group_descriptor() noexcept(false) {} // NOLINT
+    virtual ~group_descriptor() = default;
     [[nodiscard]] virtual bool owned(const id_type) const noexcept {
         return false;
     }

+ 1 - 1
src/entt/entity/mixin.hpp

@@ -182,7 +182,7 @@ public:
           update{std::move(other.update), allocator} {}
 
     /*! @brief Default destructor. */
-    ~basic_sigh_mixin() noexcept(noexcept(std::declval<underlying_type>().~underlying_type())) override = default;
+    ~basic_sigh_mixin() override = default;
 
     /**
      * @brief Default copy assignment operator, deleted on purpose.

+ 1 - 1
src/entt/entity/runtime_view.hpp

@@ -175,7 +175,7 @@ public:
           filter{std::move(other.filter), allocator} {}
 
     /*! @brief Default destructor. */
-    ~basic_runtime_view() noexcept = default;
+    ~basic_runtime_view() = default;
 
     /**
      * @brief Default copy assignment operator.

+ 3 - 3
src/entt/entity/snapshot.hpp

@@ -68,7 +68,7 @@ public:
     basic_snapshot(basic_snapshot &&) noexcept = default;
 
     /*! @brief Default destructor. */
-    ~basic_snapshot() noexcept = default;
+    ~basic_snapshot() = default;
 
     /**
      * @brief Default copy assignment operator, deleted on purpose.
@@ -200,7 +200,7 @@ public:
     basic_snapshot_loader(basic_snapshot_loader &&) noexcept = default;
 
     /*! @brief Default destructor. */
-    ~basic_snapshot_loader() noexcept = default;
+    ~basic_snapshot_loader() = default;
 
     /**
      * @brief Default copy assignment operator, deleted on purpose.
@@ -380,7 +380,7 @@ public:
     basic_continuous_loader(basic_continuous_loader &&) noexcept = default;
 
     /*! @brief Default destructor. */
-    ~basic_continuous_loader() noexcept = default;
+    ~basic_continuous_loader() = default;
 
     /**
      * @brief Default copy assignment operator, deleted on purpose.

+ 1 - 1
src/entt/entity/sparse_set.hpp

@@ -457,7 +457,7 @@ public:
     }
 
     /*! @brief Default destructor. */
-    virtual ~basic_sparse_set() noexcept(false) {
+    virtual ~basic_sparse_set() {
         release_sparse_pages();
     }
 

+ 3 - 3
src/entt/entity/storage.hpp

@@ -461,7 +461,7 @@ public:
     }
 
     /*! @brief Default destructor. */
-    ~basic_storage() noexcept(false) override {
+    ~basic_storage() override {
         shrink_to_size(0u);
     }
 
@@ -839,7 +839,7 @@ public:
         : base_type{std::move(other), allocator} {}
 
     /*! @brief Default destructor. */
-    ~basic_storage() noexcept(false) override = default;
+    ~basic_storage() override = default;
 
     /**
      * @brief Default copy assignment operator, deleted on purpose.
@@ -1061,7 +1061,7 @@ public:
           placeholder{other.placeholder} {}
 
     /*! @brief Default destructor. */
-    ~basic_storage() noexcept(false) override = default;
+    ~basic_storage() override = default;
 
     /**
      * @brief Default copy assignment operator, deleted on purpose.