From c1672ac49e394b724184042a4a0458670f455dea Mon Sep 17 00:00:00 2001
From: Siraaj Khandkar <siraaj@khandkar.net>
Date: Tue, 12 Aug 2014 15:51:52 -0400
Subject: [PATCH] Fix hope_kv_list:[iter/2, filter/2].

---
 src/hope_kv_list.erl | 23 ++++++++++++-----------
 1 file changed, 12 insertions(+), 11 deletions(-)

diff --git a/src/hope_kv_list.erl b/src/hope_kv_list.erl
index 12dbcd2..431932f 100644
--- a/src/hope_kv_list.erl
+++ b/src/hope_kv_list.erl
@@ -51,17 +51,18 @@ update(T, K, F) ->
     % TODO: Eliminate the 2nd lookup.
     set(T, K, V2).
 
-iter(T, Map1) ->
-    Map2 = lift_map_into_list(Map1),
-    lists:foreach(Map2, T).
+iter(T, F1) ->
+    F2 = lift(F1),
+    lists:foreach(F2, T).
 
-map(T, Map1) ->
-    Map2 = lift_map_into_list(Map1),
-    lists:map(Map2, T).
+map(T, F1) ->
+    F2 = lift(F1),
+    F3 = fun ({K, _}=X) -> {K, F2(X)} end,
+    lists:map(F3, T).
 
-filter(T, Map1) ->
-    Map2 = lift_map_into_list(Map1),
-    lists:filter(Map2, T).
+filter(T, F1) ->
+    F2 = lift(F1),
+    lists:filter(F2, T).
 
 fold(T, F1, Accumulator) ->
     F2 = fun ({K, V}, Acc) -> F1(K, V, Acc) end,
@@ -79,5 +80,5 @@ of_kv_list(List) ->
 %% Helpers
 %% ============================================================================
 
-lift_map_into_list(Map) ->
-    fun ({K, V}) -> {K, Map(K, V)} end.
+lift(F) ->
+    fun ({K, V}) -> F(K, V) end.
-- 
2.20.1