some small updates

Author: MAIPA01

include/events/mstd/event_handler.hpp

@@ -14,7 +14,7 @@ namespace mstd {
 
 	template<typename... Args>
 	class event_handler {
-		using id_type = int64_t;
+		using id_type = size_t;
 		using event_action_type = action<Args...>;
 		using events_type = std::unordered_map<id_type, event_action_type>;
 		using id_manager_type = id_manager<id_type>;
@@ -24,23 +24,23 @@ namespace mstd {
 		id_manager_type _ids = {};
 
 	public:
-		event_handler() = default;
-		virtual ~event_handler() = default;
+		event_handler() noexcept = default;
+		virtual ~event_handler() noexcept = default;
 
 #if _HAS_CXX20 && _MSTD_ENABLE_CXX20
 		template<event_action_func<event_action_type> F>
 #else
 		template<class F, std::enable_if_t<mstd::is_same_function_v<F, event_action_type>, bool> = true>
 #endif
-		constexpr id_type add_callback(const F& callback) {
+		[[nodiscard]] constexpr id_type add_callback(const F& callback) {
 			id_type id = _ids.get_next_id();
-			if (id == -1) return id;
+			if (id == id_manager_type::bad_id()) return id;
 
 			_events[id] = callback;
 			return id;
 		}
 
-		bool remove_callback(const id_type& callbackId) {
+		[[nodiscard]] constexpr bool remove_callback(const id_type& callbackId) {
 			auto& itr = _events.find(callbackId);
 			if (itr == _events.end()) {
 				return false;
@@ -62,15 +62,15 @@ namespace mstd {
 		}
 
 #if _HAS_CXX20 && _MSTD_ENABLE_CXX20
-		template<event_action_func F>
+		template<eevent_action_func<event_action_type> F>
 #else
 		template<class F, std::enable_if_t<mstd::is_same_function_v<F, event_action_type>, bool> = true>
 #endif
-		constexpr id_type operator+=(const F& callback) {
+		[[nodiscard]] constexpr id_type operator+=(const F& callback) {
 			return add_callback(callback);
 		}
 
-		constexpr bool operator-=(id_type callbackId) {
+		[[nodiscard]] constexpr bool operator-=(id_type callbackId) {
 			return remove_callback(callbackId);
 		}
 

include/main/mstd/mstd_config.hpp

@@ -11,7 +11,7 @@
 #pragma region VERSION
 #define MSTD_VERSION_MAJOR 1
 #define MSTD_VERSION_MINOR 2
-#define MSTD_VERSION_PATCH 0
+#define MSTD_VERSION_PATCH 1
 
 #define _MSTD_STRINGIFY_HELPER(x) #x
 
@@ -30,7 +30,7 @@
 #pragma endregion
 
 #pragma region LAST_UPDATE
-#define MSTD_LAST_UPDATE_DAY 4
+#define MSTD_LAST_UPDATE_DAY 15
 #define MSTD_LAST_UPDATE_MONTH 10
 #define MSTD_LAST_UPDATE_YEAR 2025
 

include/utils/mstd/id_manager.hpp

@@ -13,9 +13,9 @@
 
 namespace mstd {
 #if _HAS_CXX20 && _MSTD_ENABLE_CXX20
-	template<signed_integral _idT>
+	template<unsigned_integral _idT>
 #else
-	template<class _idT, std::enable_if_t<std::is_signed_v<_idT>, bool> = true>
+	template<class _idT = size_t, std::enable_if_t<std::is_unsigned_v<_idT>, bool> = true>
 #endif
 	class id_manager {
 	public:
@@ -27,7 +27,7 @@ namespace mstd {
 
 		static constexpr id_type _maxIds = std::numeric_limits<id_type>::max();
 
-		void _update_removed_ids() {
+		constexpr void _update_removed_ids() {
 			if (_removedIds.empty()) return;
 
 			auto& last = --_removedIds.end();
@@ -42,25 +42,25 @@ namespace mstd {
 		}
 
 	public:
-		id_manager() = default;
-		virtual ~id_manager() = default;
+		id_manager() noexcept = default;
+		virtual ~id_manager() noexcept = default;
 
-		id_type get_next_id() {
+		[[nodiscard]] constexpr id_type get_next_id() {
 			if (!_removedIds.empty()) {
 				const id_type id = *_removedIds.begin();
 				_removedIds.erase(id);
 				return id;
 			}
 
-			if (_nextId == _maxIds) return id_type(-1);
+			if (_nextId == _maxIds) return bad_id();
 
 			const id_type id = _nextId;
 			++_nextId;
 			return _nextId;
 		}
 
-		bool return_id(const id_type& id) {
-			if (id == -1 || id >= _nextId || _removedIds.find(id) != _removedIds.end()) {
+		[[nodiscard]] constexpr bool return_id(const id_type& id) {
+			if (id == bad_id() || id >= _nextId || _removedIds.find(id) != _removedIds.end()) {
 				return false;
 			}
 
@@ -69,22 +69,26 @@ namespace mstd {
 			return true;
 		}
 
-		void reset() {
+		void reset() noexcept {
 			_nextId = 0;
 			_removedIds.clear();
 		}
 
-		static constexpr id_type max_ids() {
+		static constexpr id_type bad_id() noexcept {
+			return static_cast<id_type>(-1);
+		}
+
+		static constexpr id_type max_ids() noexcept {
 			return _maxIds;
 		}
 
-		static constexpr id_type last_id() {
+		static constexpr id_type last_id() noexcept {
 			return _maxIds - 1;
 		}
 	};
 
-	using id8_manager = id_manager<int8_t>;
-	using id16_manager = id_manager<int16_t>;
-	using id32_manager = id_manager<int32_t>;
-	using id64_manager = id_manager<int64_t>;
+	using id8_manager = id_manager<uint8_t>;
+	using id16_manager = id_manager<uint16_t>;
+	using id32_manager = id_manager<uint32_t>;
+	using id64_manager = id_manager<uint64_t>;
 }