From aeb1ef3b125921e814ad83722a728ff02aebf94e Mon Sep 17 00:00:00 2001
From: Gary Talent <gary@drinkingtea.net>
Date: Fri, 10 May 2024 23:58:07 -0500
Subject: [PATCH] [ox/std] Cleanup SmallMap, make it easier to make potential
 changes

---
 deps/ox/src/ox/std/smallmap.hpp | 18 +++++++-----------
 1 file changed, 7 insertions(+), 11 deletions(-)

diff --git a/deps/ox/src/ox/std/smallmap.hpp b/deps/ox/src/ox/std/smallmap.hpp
index db2fa3eb..f8b58c4d 100644
--- a/deps/ox/src/ox/std/smallmap.hpp
+++ b/deps/ox/src/ox/std/smallmap.hpp
@@ -29,11 +29,12 @@ class SmallMap {
 			K key = {};
 			T value{};
 		};
+		using PairVector = Vector<Pair>;
 		Vector<K> m_keys;
-		Vector<Pair> m_pairs;
+		PairVector m_pairs;
 
 	public:
-		explicit constexpr SmallMap(std::size_t size = 127);
+		constexpr SmallMap() = default;
 
 		constexpr SmallMap(SmallMap const&other);
 
@@ -68,17 +69,13 @@ class SmallMap {
 
 	private:
 		template<typename KK>
-		constexpr Pair const&access(Vector<Pair> const&pairs, KK const&key, bool &isNew) const;
+		constexpr Pair const&access(PairVector const&pairs, KK const&key, bool &isNew) const;
 
 		template<typename KK>
-		constexpr Pair &access(Vector<Pair> &pairs, KK const&key, bool &isNew);
+		constexpr Pair &access(PairVector &pairs, KK const&key, bool &isNew);
 
 };
 
-template<typename K, typename T>
-constexpr SmallMap<K, T>::SmallMap(std::size_t size): m_pairs(size) {
-}
-
 template<typename K, typename T>
 constexpr SmallMap<K, T>::SmallMap(SmallMap<K, T> const&other) {
 	m_pairs = other.m_pairs;
@@ -188,13 +185,12 @@ constexpr Vector<K> const&SmallMap<K, T>::keys() const noexcept {
 template<typename K, typename T>
 constexpr void SmallMap<K, T>::clear() {
 	m_pairs.clear();
-	m_pairs.resize(127);
 }
 
 template<typename K, typename T>
 template<typename KK>
 constexpr typename SmallMap<K, T>::Pair const&SmallMap<K, T>::access(
-		Vector<Pair> const&pairs, KK const&k, bool &isNew) const {
+		PairVector const&pairs, KK const&k, bool &isNew) const {
 	for (auto const&p : pairs) {
 		if (p.key == k) {
 			isNew = false;
@@ -209,7 +205,7 @@ constexpr typename SmallMap<K, T>::Pair const&SmallMap<K, T>::access(
 template<typename K, typename T>
 template<typename KK>
 constexpr typename SmallMap<K, T>::Pair &SmallMap<K, T>::access(
-		Vector<Pair> &pairs, KK const&k, bool &isNew) {
+		PairVector &pairs, KK const&k, bool &isNew) {
 	for (auto &p : pairs) {
 		if (p.key == k) {
 			isNew = false;