Add some TODO notes.
authorSiraaj Khandkar <siraaj@khandkar.net>
Tue, 12 Aug 2014 19:03:23 +0000 (15:03 -0400)
committerSiraaj Khandkar <siraaj@khandkar.net>
Tue, 12 Aug 2014 19:05:25 +0000 (15:05 -0400)
src/hope_dictionary.erl
src/hope_kv_list.erl
test/hope_kv_list_SUITE.erl

index 2e3a59d..5494f39 100644 (file)
@@ -33,6 +33,7 @@
 -callback iter(t(K, V), fun((K, V) -> ok)) ->
     ok.
 
+%% TODO: Decide if validation is to be done. If yes - wrap in hope_result:t/1
 -callback of_kv_list([{K, V}]) ->
     t(K, V).
 
index 0b80a4a..12dbcd2 100644 (file)
@@ -48,6 +48,7 @@ set(T, K, V) ->
 update(T, K, F) ->
     V1Opt = get(T, K),
     V2 = F(V1Opt),
+    % TODO: Eliminate the 2nd lookup.
     set(T, K, V2).
 
 iter(T, Map1) ->
@@ -70,6 +71,7 @@ to_kv_list(T) ->
     T.
 
 of_kv_list(List) ->
+    % TODO: Decide if validation is to be done here. Do so if yes.
     List.
 
 
index 98c5489..d99e669 100644 (file)
@@ -20,6 +20,9 @@
 %% Common Test callbacks
 %% ============================================================================
 
+%% TODO: Make tests generic for any dictionary.
+%% TODO: Each group shold test a type of dictionary against the generic cases.
+
 all() ->
     [{group, ?GROUP_KV_LIST}].
 
This page took 0.041 seconds and 4 git commands to generate.