status-im/status-mobile
2
0
Fork 0
mirror of https://github.com/status-im/status-mobile.git synced 2025-02-06 05:45:03 +00:00
Code Issues Projects Releases Wiki Activity

[performance] replace reduce by reduce-kv

Browse Source 
For associative collections, using reduce-kv should
give a small performance gain over reduce as there is
no allocation of tuples for key/value pairs

Signed-off-by: yenda <eric@status.im>
This commit is contained in:
yenda 2019-05-17 00:00:22 +02:00
parent 29eeb140b9
commit 70e889a03a
No known key found for this signature in database
GPG Key ID: 0095623C0069DCE6
2 changed files with 12 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/status_im/chat/db.cljs
View File

@ -44,9 +44,11 @@
(defn active-chats
[contacts chats {:keys [public-key]}]
(reduce (fn [acc [chat-id {:keys [is-active] :as chat}]]
(reduce-kv (fn [acc chat-id {:keys [is-active] :as chat}]
(if is-active
(assoc acc chat-id (enrich-active-chat contacts chat public-key))
(assoc acc
chat-id
(enrich-active-chat contacts chat public-key))
acc))
{}
chats))

2
src/status_im/contact/db.cljs
View File

@ -177,7 +177,7 @@
(defn enrich-contacts
[contacts]
(reduce (fn [acc [public-key contact]]
(reduce-kv (fn [acc public-key contact]
(assoc acc public-key (enrich-contact contact)))
{}
contacts))