From ae16d496b9f2b245167a8af3fdab3e124364047d Mon Sep 17 00:00:00 2001
From: Max Horn
Date: Sun, 6 Sep 2009 12:59:07 +0000
Subject: COMMON: HashMap::getVal now allows specifying a default value.

A new variant of HashMap::getVal with a second 'default value' parameter
has been added. This helps avoid many contains() + getVal() combos
(which incur double lookup penalty), and is much lighter than using
find() (which has to create an iterator).

svn-id: r43983
---
 test/common/hashmap.h | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

(limited to 'test')

diff --git a/test/common/hashmap.h b/test/common/hashmap.h
index 6476005eaf..03db5518c0 100644
--- a/test/common/hashmap.h
+++ b/test/common/hashmap.h
@@ -86,6 +86,24 @@ class HashMapTestSuite : public CxxTest::TestSuite
 		TS_ASSERT_EQUALS(container[4], 96);
 	}
 
+	void test_lookup_with_default() {
+		Common::HashMap<int, int> container;
+		container[0] = 17;
+		container[1] = -1;
+		container[2] = 45;
+		container[3] = 12;
+		container[4] = 96;
+
+		// We take a const ref now to ensure that the map
+		// is not modified by getVal.
+		const Common::HashMap<int, int> &containerRef = container;
+
+		TS_ASSERT_EQUALS(containerRef.getVal(0), 17);
+		TS_ASSERT_EQUALS(containerRef.getVal(17), 0);
+		TS_ASSERT_EQUALS(containerRef.getVal(0, -10), 17);
+		TS_ASSERT_EQUALS(containerRef.getVal(17, -10), -10);
+	}
+
 	void test_iterator_begin_end() {
 		Common::HashMap<int, int> container;
 
-- 
cgit v1.2.3