move ObjectMap::empty() into header and make noexcept

freeorion · Aug 2, 2022 · 59d485c · 59d485c
1 parent 20dd2ed
commit 59d485c
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/universe/ObjectMap.cpp b/universe/ObjectMap.cpp
@@ -219,9 +219,6 @@ ObjectMap* ObjectMap::Clone(const Universe& universe, int empire_id) const {
     return result.release();
 }
 
-bool ObjectMap::empty() const
-{ return m_objects.empty(); }
-
 int ObjectMap::HighestObjectID() const {
     if (m_objects.empty())
         return INVALID_OBJECT_ID;

diff --git a/universe/ObjectMap.h b/universe/ObjectMap.h
@@ -42,7 +42,7 @@ class FO_COMMON_API ObjectMap {
     [[nodiscard]] std::size_t size() const;
 
     /** Returns true if this ObjectMap contains no objects */
-    [[nodiscard]] bool empty() const;
+    [[nodiscard]] bool empty() const noexcept { return m_objects.empty(); };
 
     /** Returns a pointer to the object of type T with ID number \a id.
       * Returns a null std::shared_ptr if none exists or the object with